Commit fd8c4170 authored by Nasko Oskov's avatar Nasko Oskov Committed by Commit Bot

Speculatively enable WebNavigationApiTest.UserAction on Windows

This test has been disabled on Windows as flaky. Running locally 100
iterations of the test passed successfully, so this CL enables it.

Bug: 662160
Change-Id: I21d192e02b157c35cb95e9aa6f5ad40425d16ff4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2141135
Commit-Queue: Nasko Oskov <nasko@chromium.org>
Auto-Submit: Nasko Oskov <nasko@chromium.org>
Reviewed-by: default avatarŁukasz Anforowicz <lukasza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#757506}
parent 687527b9
...@@ -323,13 +323,7 @@ IN_PROC_BROWSER_TEST_F(WebNavigationApiTest, FilteredTest) { ...@@ -323,13 +323,7 @@ IN_PROC_BROWSER_TEST_F(WebNavigationApiTest, FilteredTest) {
ASSERT_TRUE(RunExtensionTest("webnavigation/filtered")) << message_; ASSERT_TRUE(RunExtensionTest("webnavigation/filtered")) << message_;
} }
// Flaky on Windows. See http://crbug.com/662160. IN_PROC_BROWSER_TEST_F(WebNavigationApiTest, UserAction) {
#if defined(OS_WIN)
#define MAYBE_UserAction DISABLED_UserAction
#else
#define MAYBE_UserAction UserAction
#endif
IN_PROC_BROWSER_TEST_F(WebNavigationApiTest, MAYBE_UserAction) {
content::IsolateAllSitesForTesting(base::CommandLine::ForCurrentProcess()); content::IsolateAllSitesForTesting(base::CommandLine::ForCurrentProcess());
ASSERT_TRUE(StartEmbeddedTestServer()); ASSERT_TRUE(StartEmbeddedTestServer());
......
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