Allow apps with background pages to request process-per-app-instance.

chrome/browser/extensions/app_process_apitest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_host.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/process_map.h"
 #include "chrome/browser/profiles/profile.h"
@@ skipping 10 matching lines @@
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/test/test_navigation_observer.h"
 #include "net/base/mock_host_resolver.h"
 
 using content::NavigationController;
 using content::WebContents;
 
-class AppApiTest : public ExtensionApiTest {
- protected:
-  // Gets the base URL for files for a specific test, making sure that it uses
-  // "localhost" as the hostname, since that is what the extent is declared
-  // as in the test apps manifests.
-  GURL GetTestBaseURL(std::string test_directory) {
-    GURL::Replacements replace_host;
-    std::string host_str("localhost");  // must stay in scope with replace_host
-    replace_host.SetHostStr(host_str);
-    GURL base_url = test_server()->GetURL(
-        "files/extensions/api_test/" + test_directory + "/");
-    return base_url.ReplaceComponents(replace_host);
-  }
-};
-
 // Simulates a page calling window.open on an URL, and waits for the navigation.
 static void WindowOpenHelper(Browser* browser,
                              RenderViewHost* opener_host,
                              const GURL& url,
                              bool newtab_process_should_equal_opener) {
   ui_test_utils::WindowedNotificationObserver observer(
       content::NOTIFICATION_LOAD_STOP,
       content::NotificationService::AllSources());
   ASSERT_TRUE(ui_test_utils::ExecuteJavaScript(
       opener_host, L"", L"window.open('" + UTF8ToWide(url.spec()) + L"');"));
@@ skipping 24 matching lines @@
       L"window.addEventListener('unload', function() {"
       L"    window.domAutomationController.send(true);"
       L"}, false);"
       L"window.location = '" + UTF8ToWide(url.spec()) + L"';",
       &result));
   ASSERT_TRUE(result);
   observer.Wait();
   EXPECT_EQ(url, contents->GetController().GetLastCommittedEntry()->GetURL());
 }
 
+class AppApiTest : public ExtensionApiTest {
+ protected:
+  // Gets the base URL for files for a specific test, making sure that it uses
+  // "localhost" as the hostname, since that is what the extent is declared
+  // as in the test apps manifests.
+  GURL GetTestBaseURL(std::string test_directory) {
+    GURL::Replacements replace_host;
+    std::string host_str("localhost");  // must stay in scope with replace_host
+    replace_host.SetHostStr(host_str);
+    GURL base_url = test_server()->GetURL(
+        "files/extensions/api_test/" + test_directory + "/");
+    return base_url.ReplaceComponents(replace_host);
+  }
+
+  // Pass flags to make testing apps easier.
+  void SetUpCommandLine(CommandLine* command_line) {
+    ExtensionApiTest::SetUpCommandLine(command_line);
+    CommandLine::ForCurrentProcess()->AppendSwitch(
+        switches::kDisablePopupBlocking);
+    CommandLine::ForCurrentProcess()->AppendSwitch(
+        switches::kAllowHTTPBackgroundPage);
+  }
+
+  // Helper function to test that independent tabs of the named app are loaded
+  // into separate processes.
+  void TestAppInstancesHelper(std::string app_name) {
+    LOG(INFO) << "Start of test.";
+
+    extensions::ProcessMap* process_map =
+        browser()->profile()->GetExtensionService()->process_map();
+
+    host_resolver()->AddRule("*", "127.0.0.1");
+    ASSERT_TRUE(test_server()->Start());
+
+    ASSERT_TRUE(LoadExtension(
+        test_data_dir_.AppendASCII(app_name)));
+
+    // Open two tabs in the app, one outside it.
+    GURL base_url = GetTestBaseURL(app_name);
+
+    // Test both opening a URL in a new tab, and opening a tab and then
+    // navigating it.  Either way, app tabs should be considered extension
+    // processes, but they have no elevated privileges and thus should not
+    // have WebUI bindings.
+    ui_test_utils::NavigateToURLWithDisposition(
+        browser(), base_url.Resolve("path1/empty.html"), NEW_FOREGROUND_TAB,
+        ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
+    LOG(INFO) << "Nav 1.";
+    EXPECT_TRUE(process_map->Contains(
+        browser()->GetWebContentsAt(1)->GetRenderProcessHost()->GetID()));
+    EXPECT_FALSE(browser()->GetWebContentsAt(1)->GetWebUI());
+
+    ui_test_utils::WindowedNotificationObserver tab_added_observer(
+        content::NOTIFICATION_TAB_ADDED,
+        content::NotificationService::AllSources());
+    browser()->NewTab();
+    tab_added_observer.Wait();
+    LOG(INFO) << "New tab.";
+    ui_test_utils::NavigateToURL(browser(),
+                                 base_url.Resolve("path2/empty.html"));
+    LOG(INFO) << "Nav 2.";
+    EXPECT_TRUE(process_map->Contains(
+        browser()->GetWebContentsAt(2)->GetRenderProcessHost()->GetID()));
+    EXPECT_FALSE(browser()->GetWebContentsAt(2)->GetWebUI());
+
+    // We should have opened 2 new extension tabs. Including the original blank
+    // tab, we now have 3 tabs. The two app tabs should not be in the same
+    // process, since they do not have the background permission.  (Thus, we
+    // want to separate them to improve responsiveness.)
+    ASSERT_EQ(3, browser()->tab_count());
+    RenderViewHost* host1 = browser()->GetWebContentsAt(1)->GetRenderViewHost();
+    RenderViewHost* host2 = browser()->GetWebContentsAt(2)->GetRenderViewHost();
+    EXPECT_NE(host1->process(), host2->process());
+
+    // Opening tabs with window.open should keep the page in the opener's
+    // process.
+    ASSERT_EQ(1u, BrowserList::GetBrowserCount(browser()->profile()));
+    WindowOpenHelper(browser(), host1,
+                    base_url.Resolve("path1/empty.html"), true);
+    LOG(INFO) << "WindowOpenHelper 1.";
+    WindowOpenHelper(browser(), host2,
+                    base_url.Resolve("path2/empty.html"), true);
+    LOG(INFO) << "End of test.";
+  }
+};
+
+// Tests that hosted apps with the background permission get a process-per-app
+// model, since all pages need to be able to script the background page.
 IN_PROC_BROWSER_TEST_F(AppApiTest, AppProcess) {
   LOG(INFO) << "Start of test.";
 
-  CommandLine::ForCurrentProcess()->AppendSwitch(
-      switches::kDisablePopupBlocking);
-
   extensions::ProcessMap* process_map =
       browser()->profile()->GetExtensionService()->process_map();
 
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
 
   ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("app_process")));
 
   LOG(INFO) << "Loaded extension.";
 
@@ skipping 87 matching lines @@
       L"window.domAutomationController.send(window.opener != null)",
       &windowOpenerValid));
   ASSERT_TRUE(windowOpenerValid);
 
   LOG(INFO) << "End of test.";
 }
 
 // Test that hosted apps without the background permission use a process per app
 // instance model, such that separate instances are in separate processes.
 IN_PROC_BROWSER_TEST_F(AppApiTest, AppProcessInstances) {
-  LOG(INFO) << "Start of test.";
+  TestAppInstancesHelper("app_process_instances");
+}
 
-  CommandLine::ForCurrentProcess()->AppendSwitch(
-      switches::kDisablePopupBlocking);
-
-  extensions::ProcessMap* process_map =
-      browser()->profile()->GetExtensionService()->process_map();
-
-  host_resolver()->AddRule("*", "127.0.0.1");
-  ASSERT_TRUE(test_server()->Start());
-
-  ASSERT_TRUE(LoadExtension(
-      test_data_dir_.AppendASCII("app_process_instances")));
-
-  // Open two tabs in the app, one outside it.
-  GURL base_url = GetTestBaseURL("app_process_instances");
-
-  // Test both opening a URL in a new tab, and opening a tab and then navigating
-  // it.  Either way, app tabs should be considered extension processes, but
-  // they have no elevated privileges and thus should not have WebUI bindings.
-  ui_test_utils::NavigateToURLWithDisposition(
-      browser(), base_url.Resolve("path1/empty.html"), NEW_FOREGROUND_TAB,
-      ui_test_utils::BROWSER_TEST_WAIT_FOR_NAVIGATION);
-  LOG(INFO) << "Nav 1.";
-  EXPECT_TRUE(process_map->Contains(
-      browser()->GetWebContentsAt(1)->GetRenderProcessHost()->GetID()));
-  EXPECT_FALSE(browser()->GetWebContentsAt(1)->GetWebUI());
-
-  ui_test_utils::WindowedNotificationObserver tab_added_observer(
-      content::NOTIFICATION_TAB_ADDED,
-      content::NotificationService::AllSources());
-  browser()->NewTab();
-  tab_added_observer.Wait();
-  LOG(INFO) << "New tab.";
-  ui_test_utils::NavigateToURL(browser(), base_url.Resolve("path2/empty.html"));
-  LOG(INFO) << "Nav 2.";
-  EXPECT_TRUE(process_map->Contains(
-      browser()->GetWebContentsAt(2)->GetRenderProcessHost()->GetID()));
-  EXPECT_FALSE(browser()->GetWebContentsAt(2)->GetWebUI());
-
-  // We should have opened 2 new extension tabs. Including the original blank
-  // tab, we now have 3 tabs. The two app tabs should not be in the same
-  // process, since they do not have the background permission.  (Thus, we want
-  // to separate them to improve responsiveness.)
-  ASSERT_EQ(3, browser()->tab_count());
-  RenderViewHost* host1 = browser()->GetWebContentsAt(1)->GetRenderViewHost();
-  RenderViewHost* host2 = browser()->GetWebContentsAt(2)->GetRenderViewHost();
-  EXPECT_NE(host1->process(), host2->process());
-
-  // Opening tabs with window.open should keep the page in the opener's process.
-  ASSERT_EQ(1u, BrowserList::GetBrowserCount(browser()->profile()));
-  WindowOpenHelper(browser(), host1,
-                   base_url.Resolve("path1/empty.html"), true);
-  LOG(INFO) << "WindowOpenHelper 1.";
-  WindowOpenHelper(browser(), host2,
-                   base_url.Resolve("path2/empty.html"), true);
-  LOG(INFO) << "End of test.";
+// Test that hosted apps with the background permission but that set
+// allow_js_access to false also use a process per app instance model.
+// Separate instances should be in separate processes.
+IN_PROC_BROWSER_TEST_F(AppApiTest, AppProcessBackgroundInstances) {
+  TestAppInstancesHelper("app_process_background_instances");
 }
 
 // Tests that bookmark apps do not use the app process model and are treated
 // like normal web pages instead.  http://crbug.com/104636.
 IN_PROC_BROWSER_TEST_F(AppApiTest, BookmarkAppGetsNormalProcess) {
-  CommandLine::ForCurrentProcess()->AppendSwitch(
-      switches::kDisablePopupBlocking);
-
   ExtensionService* service = browser()->profile()->GetExtensionService();
   extensions::ProcessMap* process_map = service->process_map();
 
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
   GURL base_url = GetTestBaseURL("app_process");
 
   // Load an app as a bookmark app.
   std::string error;
   scoped_refptr<const Extension> extension(extension_file_util::LoadExtension(
@@ skipping 60 matching lines @@
 // 3. page2 redirects back to a page in the app
 // The final navigation should end up in the app process.
 // See http://crbug.com/61757
 // This test doesn't complete on WebKit Win (dbg). See crbug.com/108853.
 #if defined(OS_WIN) && !defined(NDEBUG)
 #define MAYBE_AppProcessRedirectBack DISABLED_AppProcessRedirectBack
 #else
 #define MAYBE_AppProcessRedirectBack AppProcessRedirectBack
 #endif
 IN_PROC_BROWSER_TEST_F(AppApiTest, MAYBE_AppProcessRedirectBack) {
-  CommandLine::ForCurrentProcess()->AppendSwitch(
-      switches::kDisablePopupBlocking);
-
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
 
   ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("app_process")));
 
   // Open two tabs in the app.
   GURL base_url = GetTestBaseURL("app_process");
 
   browser()->NewTab();
   ui_test_utils::NavigateToURL(browser(), base_url.Resolve("path1/empty.html"));
@@ skipping 20 matching lines @@
   EXPECT_EQ("/files/extensions/api_test/app_process/path1/empty.html",
             browser()->GetWebContentsAt(2)->GetController().
                 GetLastCommittedEntry()->GetURL().path());
   EXPECT_EQ(browser()->GetWebContentsAt(1)->GetRenderProcessHost(),
             browser()->GetWebContentsAt(2)->GetRenderProcessHost());
 }
 
 // Ensure that reloading a URL after installing or uninstalling it as an app
 // correctly swaps the process.  (http://crbug.com/80621)
 IN_PROC_BROWSER_TEST_F(AppApiTest, ReloadIntoAppProcess) {
-  CommandLine::ForCurrentProcess()->AppendSwitch(
-      switches::kDisablePopupBlocking);
-
   extensions::ProcessMap* process_map =
       browser()->profile()->GetExtensionService()->process_map();
 
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
 
   // The app under test acts on URLs whose host is "localhost",
   // so the URLs we navigate to must have host "localhost".
   GURL base_url = GetTestBaseURL("app_process");
 
@@ skipping 67 matching lines @@
   EXPECT_FALSE(process_map->Contains(
       contents->GetRenderProcessHost()->GetID()));
 }
 
 // Tests that if we have a non-app process (path3/container.html) that has an
 // iframe with  a URL in the app's extent (path1/iframe.html), then opening a
 // link from that iframe to a new window to a URL in the app's extent (path1/
 // empty.html) results in the new window being in an app process. See
 // http://crbug.com/89272 for more details.
 IN_PROC_BROWSER_TEST_F(AppApiTest, OpenAppFromIframe) {
-  CommandLine::ForCurrentProcess()->AppendSwitch(
-      switches::kDisablePopupBlocking);
-
   extensions::ProcessMap* process_map =
       browser()->profile()->GetExtensionService()->process_map();
 
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
 
   GURL base_url = GetTestBaseURL("app_process");
 
   // Load app and start URL (not in the app).
   const Extension* app =
@@ skipping 75 matching lines @@
 }
 
 // Tests that if we have an app process (path1/container.html) with a non-app
 // iframe (path3/iframe.html), then opening a link from that iframe to a new
 // window to a same-origin non-app URL (path3/empty.html) should keep the window
 // in the app process.
 // This is in contrast to OpenAppFromIframe, since here the popup will not be
 // missing special permissions and should be scriptable from the iframe.
 // See http://crbug.com/92669 for more details.
 IN_PROC_BROWSER_TEST_F(AppApiTest, OpenWebPopupFromWebIframe) {
-  CommandLine::ForCurrentProcess()->AppendSwitch(
-      switches::kDisablePopupBlocking);
-
   extensions::ProcessMap* process_map =
       browser()->profile()->GetExtensionService()->process_map();
 
   host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(test_server()->Start());
 
   GURL base_url = GetTestBaseURL("app_process");
 
   // Load app and start URL (in the app).
   const Extension* app =
@@ skipping 66 matching lines @@
           &browser()->GetSelectedTabContentsWrapper()->web_contents()->
               GetController()));
   browser()->Reload(CURRENT_TAB);
   observer.Wait();
   ASSERT_TRUE(ui_test_utils::ExecuteJavaScriptAndExtractBool(
       contents->GetRenderViewHost(), L"",
       L"window.domAutomationController.send(chrome.app.isInstalled)",
       &is_installed));
   ASSERT_TRUE(is_installed);
 }