Commit d1747b9e authored by Devlin Cronin's avatar Devlin Cronin Committed by Commit Bot

[Extensions] Re-enable IncognitoApiTest.IncognitoYesScript

This test was disabled for being flaky on XP.  XP is dead.  The test now
passes 100 runs on Windows.

Bug: 248821
Change-Id: Icda344e6c7c26b5046cbf1d51929cd200792ca6f
Reviewed-on: https://chromium-review.googlesource.com/663607Reviewed-by: default avatarIstiaque Ahmed <lazyboy@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501523}
parent 928b9963
...@@ -53,14 +53,7 @@ IN_PROC_BROWSER_TEST_F(IncognitoApiTest, IncognitoNoScript) { ...@@ -53,14 +53,7 @@ IN_PROC_BROWSER_TEST_F(IncognitoApiTest, IncognitoNoScript) {
EXPECT_TRUE(result); EXPECT_TRUE(result);
} }
#if defined(OS_WIN) IN_PROC_BROWSER_TEST_F(IncognitoApiTest, IncognitoYesScript) {
// This test is very flaky on XP. http://crbug.com/248821
#define MAYBE_IncognitoYesScript DISABLED_IncognitoYesScript
#else
#define MAYBE_IncognitoYesScript IncognitoYesScript
#endif
IN_PROC_BROWSER_TEST_F(IncognitoApiTest, MAYBE_IncognitoYesScript) {
// Load a dummy extension. This just tests that we don't regress a // Load a dummy extension. This just tests that we don't regress a
// crash fix when multiple incognito- and non-incognito-enabled extensions // crash fix when multiple incognito- and non-incognito-enabled extensions
// are mixed. // are mixed.
......
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