Index: chrome/browser/extensions/window_open_apitest.cc |
diff --git a/chrome/browser/extensions/window_open_apitest.cc b/chrome/browser/extensions/window_open_apitest.cc |
index 1b31d161937be5c301f345d7440ff79ebf3ec99e..249386fa768f3d7597469b6c7c996a95c4d17f45 100644 |
--- a/chrome/browser/extensions/window_open_apitest.cc |
+++ b/chrome/browser/extensions/window_open_apitest.cc |
@@ -44,11 +44,18 @@ using content::OpenURLParams; |
using content::Referrer; |
using content::WebContents; |
+class WindowOpenApiTest : public ExtensionApiTest { |
+ void SetUpOnMainThread() override { |
+ ExtensionApiTest::SetUpOnMainThread(); |
+ host_resolver()->AddRule("*", "127.0.0.1"); |
+ } |
+}; |
+ |
// The test uses the chrome.browserAction.openPopup API, which requires that the |
// window can automatically be activated. |
// See comments at BrowserActionInteractiveTest::ShouldRunPopupTest |
// Fails flakily on all platforms. https://crbug.com/477691 |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_WindowOpen) { |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, DISABLED_WindowOpen) { |
extensions::ResultCatcher catcher; |
ASSERT_TRUE(LoadExtensionIncognito(test_data_dir_ |
.AppendASCII("window_open").AppendASCII("spanning"))); |
@@ -93,8 +100,7 @@ bool WaitForTabsAndPopups(Browser* browser, |
(num_popups == num_popups_seen)); |
} |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, BrowserIsApp) { |
- host_resolver()->AddRule("a.com", "127.0.0.1"); |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, BrowserIsApp) { |
ASSERT_TRUE(StartEmbeddedTestServer()); |
ASSERT_TRUE(LoadExtension( |
test_data_dir_.AppendASCII("window_open").AppendASCII("browser_is_app"))); |
@@ -109,7 +115,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, BrowserIsApp) { |
} |
} |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowOpenPopupDefault) { |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, WindowOpenPopupDefault) { |
ASSERT_TRUE(StartEmbeddedTestServer()); |
ASSERT_TRUE(LoadExtension( |
test_data_dir_.AppendASCII("window_open").AppendASCII("popup"))); |
@@ -119,7 +125,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowOpenPopupDefault) { |
EXPECT_TRUE(WaitForTabsAndPopups(browser(), num_tabs, num_popups)); |
} |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowOpenPopupIframe) { |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, WindowOpenPopupIframe) { |
base::FilePath test_data_dir; |
PathService::Get(chrome::DIR_TEST_DATA, &test_data_dir); |
embedded_test_server()->ServeFilesFromDirectory(test_data_dir); |
@@ -132,7 +138,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowOpenPopupIframe) { |
EXPECT_TRUE(WaitForTabsAndPopups(browser(), num_tabs, num_popups)); |
} |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowOpenPopupLarge) { |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, WindowOpenPopupLarge) { |
ASSERT_TRUE(StartEmbeddedTestServer()); |
ASSERT_TRUE(LoadExtension( |
test_data_dir_.AppendASCII("window_open").AppendASCII("popup_large"))); |
@@ -143,7 +149,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowOpenPopupLarge) { |
EXPECT_TRUE(WaitForTabsAndPopups(browser(), num_tabs, num_popups)); |
} |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowOpenPopupSmall) { |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, WindowOpenPopupSmall) { |
ASSERT_TRUE(StartEmbeddedTestServer()); |
ASSERT_TRUE(LoadExtension( |
test_data_dir_.AppendASCII("window_open").AppendASCII("popup_small"))); |
@@ -161,8 +167,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowOpenPopupSmall) { |
#else |
#define MAYBE_PopupBlockingExtension PopupBlockingExtension |
#endif |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MAYBE_PopupBlockingExtension) { |
- host_resolver()->AddRule("*", "127.0.0.1"); |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, MAYBE_PopupBlockingExtension) { |
ASSERT_TRUE(StartEmbeddedTestServer()); |
ASSERT_TRUE(LoadExtension( |
@@ -172,8 +177,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MAYBE_PopupBlockingExtension) { |
EXPECT_TRUE(WaitForTabsAndPopups(browser(), 5, 3)); |
} |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, PopupBlockingHostedApp) { |
- host_resolver()->AddRule("*", "127.0.0.1"); |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, PopupBlockingHostedApp) { |
ASSERT_TRUE(embedded_test_server()->Start()); |
ASSERT_TRUE(LoadExtension( |
@@ -206,11 +210,11 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, PopupBlockingHostedApp) { |
EXPECT_TRUE(WaitForTabsAndPopups(browser(), 3, 1)); |
} |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, WindowArgumentsOverflow) { |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, WindowArgumentsOverflow) { |
ASSERT_TRUE(RunExtensionTest("window_open/argument_overflow")) << message_; |
} |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_WindowOpener) { |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, DISABLED_WindowOpener) { |
ASSERT_TRUE(RunExtensionTest("window_open/opener")) << message_; |
} |
@@ -222,7 +226,7 @@ IN_PROC_BROWSER_TEST_F(ExtensionApiTest, DISABLED_WindowOpener) { |
#endif |
// Ensure that the width and height properties of a window opened with |
// chrome.windows.create match the creation parameters. See crbug.com/173831. |
-IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MAYBE_WindowOpenSized) { |
+IN_PROC_BROWSER_TEST_F(WindowOpenApiTest, MAYBE_WindowOpenSized) { |
ASSERT_TRUE(RunExtensionTest("window_open/window_size")) << message_; |
EXPECT_TRUE(WaitForTabsAndPopups(browser(), 0, 1)); |
} |