Commit 5905c3bc authored by Kelvin Jiang's avatar Kelvin Jiang Committed by Commit Bot

[Sheriff] Disable StartupTracingThrottle test

This CL disables
ChromeTracingDelegateBrowserTestOnStartup.StartupTracingThrottle as it
is failing on multiple mac builders.

Bug: 1134793
TBR: eseckler@chromium.org
Change-Id: I72f1702bdb5613d4c86f71adb0c66283ccdb3ea6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2446994
Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org>
Reviewed-by: default avatarKelvin Jiang <kelvinjiang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#813467}
parent 6c1ea5c8
...@@ -285,8 +285,10 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup, ...@@ -285,8 +285,10 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup,
content::BackgroundTracingManager::GetInstance()->HasActiveScenario()); content::BackgroundTracingManager::GetInstance()->HasActiveScenario());
} }
// TODO(crbug.com/1134793): Test is disabled after failing on multiple mac
// builders.
IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup, IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup,
PRE_PRE_StartupTracingThrottle) { DISABLED_PRE_PRE_StartupTracingThrottle) {
// This test exists just to make sure the browser is created at least once and // This test exists just to make sure the browser is created at least once and
// so a default profile is created. Then, the next time the browser is // so a default profile is created. Then, the next time the browser is
// created, kMetricsReportingEnabled is explicitly read from the profile and // created, kMetricsReportingEnabled is explicitly read from the profile and
...@@ -294,7 +296,7 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup, ...@@ -294,7 +296,7 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup,
} }
IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup, IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup,
PRE_StartupTracingThrottle) { DISABLED_PRE_StartupTracingThrottle) {
EXPECT_TRUE( EXPECT_TRUE(
content::BackgroundTracingManager::GetInstance()->HasActiveScenario()); content::BackgroundTracingManager::GetInstance()->HasActiveScenario());
...@@ -308,7 +310,7 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup, ...@@ -308,7 +310,7 @@ IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup,
// https://crbug.com/832981: The test is reenabled to check if flakiness still // https://crbug.com/832981: The test is reenabled to check if flakiness still
// exists. // exists.
IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup, IN_PROC_BROWSER_TEST_F(ChromeTracingDelegateBrowserTestOnStartup,
StartupTracingThrottle) { DISABLED_StartupTracingThrottle) {
// The startup scenario should *not* be started, since not enough // The startup scenario should *not* be started, since not enough
// time has elapsed since the last upload (set in the PRE_ above). // time has elapsed since the last upload (set in the PRE_ above).
EXPECT_FALSE( EXPECT_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