Commit a9ed11b6 authored by Yi Gu's avatar Yi Gu Committed by Chromium LUCI CQ

[Sheriff] Disable LoadTimingBrowserTest.HTTPS on more platforms

The test is flaky on all platforms:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=LoadTimingBrowserTest.HTTPS&testType=browser_tests

TBR=ryansturm@chromium.org

Bug: 1128033
Change-Id: I5de88f598d93ff43f06a617a9713d95375698775
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2633684
Commit-Queue: Yi Gu <yigu@chromium.org>
Auto-Submit: Yi Gu <yigu@chromium.org>
Reviewed-by: default avatarFabrice de Gans-Riberi <fdegans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844202}
parent 5ef0e3bb
......@@ -133,13 +133,8 @@ IN_PROC_BROWSER_TEST_F(LoadTimingBrowserTest, HTTP) {
EXPECT_EQ(navigation_deltas.ssl_start, -1);
}
// TODO(crbug.com/1128033): Flaky on ChromeOS and Mac.
#if BUILDFLAG(IS_CHROMEOS_ASH) || defined(OS_MAC)
#define MAYBE_HTTPS DISABLED_HTTPS
#else
#define MAYBE_HTTPS HTTPS
#endif
IN_PROC_BROWSER_TEST_F(LoadTimingBrowserTest, MAYBE_HTTPS) {
// TODO(crbug.com/1128033): Flaky on all platforms
IN_PROC_BROWSER_TEST_F(LoadTimingBrowserTest, DISABLED_HTTPS) {
net::EmbeddedTestServer https_server(net::EmbeddedTestServer::TYPE_HTTPS);
https_server.AddDefaultHandlers();
ASSERT_TRUE(https_server.Start());
......
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