Commit 6a5318fb authored by Pavol Marko's avatar Pavol Marko Committed by Commit Bot

Re-enable MergeSessionTest.Throttle

The underlying issue that has been causing many flakes here
(https://crbug.com/1005082) has been fixed, so re-enable the
browsertest.

Bug: 1005084
Change-Id: I0147578787ca11d8ddbfd49158dfc5eda5bf8a36
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1821482Reviewed-by: default avatarChris Mumford <cmumford@google.com>
Reviewed-by: default avatarAchuith Bhandarkar <achuith@chromium.org>
Reviewed-by: default avatarPavol Marko <pmarko@chromium.org>
Commit-Queue: Pavol Marko <pmarko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699723}
parent 56e56ad6
...@@ -17,7 +17,6 @@ ...@@ -17,7 +17,6 @@
#include "base/strings/stringprintf.h" #include "base/strings/stringprintf.h"
#include "base/synchronization/waitable_event.h" #include "base/synchronization/waitable_event.h"
#include "base/task/post_task.h" #include "base/task/post_task.h"
#include "build/build_config.h"
#include "chrome/browser/browser_process.h" #include "chrome/browser/browser_process.h"
#include "chrome/browser/chrome_notification_types.h" #include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/chromeos/login/existing_user_controller.h" #include "chrome/browser/chromeos/login/existing_user_controller.h"
...@@ -1030,13 +1029,7 @@ IN_PROC_BROWSER_TEST_P(MergeSessionTest, PageThrottle) { ...@@ -1030,13 +1029,7 @@ IN_PROC_BROWSER_TEST_P(MergeSessionTest, PageThrottle) {
DVLOG(1) << "Loaded page at the end : " << title; DVLOG(1) << "Loaded page at the end : " << title;
} }
// TODO(crbug.com/1005084) Disabled on Chrome OS due to timeouts IN_PROC_BROWSER_TEST_P(MergeSessionTest, Throttle) {
#if defined(OS_CHROMEOS)
#define MAYBE_XHRThrottle DISABLED_XHRThrottle
#else
#define MAYBE_XHRThrottle XHRThrottle
#endif
IN_PROC_BROWSER_TEST_P(MergeSessionTest, MAYBE_XHRThrottle) {
StartNewUserSession(/*wait_for_merge=*/false, StartNewUserSession(/*wait_for_merge=*/false,
/*is_under_advanced_protection=*/false); /*is_under_advanced_protection=*/false);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment