| 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));
|
| }
|
|
|