Update some host_resolver()->AddRules in chrome/browser.

chrome/browser/extensions/process_management_browsertest.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 <stddef.h>
 
 #include "base/strings/utf_string_conversions.h"
 #include "build/build_config.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/extension_apitest.h"
@@ skipping 28 matching lines @@
 namespace {
 
 class ProcessManagementTest : public ExtensionBrowserTest {
  private:
   // This is needed for testing isolated apps, which are still experimental.
   void SetUpCommandLine(base::CommandLine* command_line) override {
     ExtensionBrowserTest::SetUpCommandLine(command_line);
     command_line->AppendSwitch(
         extensions::switches::kEnableExperimentalExtensionApis);
   }
+
+  void SetUpOnMainThread() override {
+    ExtensionBrowserTest::SetUpOnMainThread();
+    host_resolver()->AddRule("*", "127.0.0.1");
+  }
 };
 
 class ChromeWebStoreProcessTest : public ExtensionBrowserTest {
  public:
   const GURL& gallery_url() { return gallery_url_; }
 
  private:
   // Overrides location of Chrome Web Store gallery to a test controlled URL.
   void SetUpCommandLine(base::CommandLine* command_line) override {
     ExtensionBrowserTest::SetUpCommandLine(command_line);
@@ skipping 22 matching lines @@
 #define MAYBE_ProcessOverflow ProcessOverflow
 #endif
 
 // Ensure that an isolated app never shares a process with WebUIs, non-isolated
 // extensions, and normal webpages.  None of these should ever comingle
 // RenderProcessHosts even if we hit the process limit.
 IN_PROC_BROWSER_TEST_F(ProcessManagementTest, MAYBE_ProcessOverflow) {
   // Set max renderers to 1 to force running out of processes.
   content::RenderProcessHost::SetMaxRendererProcessCount(1);
 
-  host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(embedded_test_server()->Start());
 
   ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("isolated_apps/app1")));
   ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("isolated_apps/app2")));
   ASSERT_TRUE(LoadExtension(test_data_dir_.AppendASCII("hosted_app")));
   ASSERT_TRUE(
       LoadExtension(test_data_dir_.AppendASCII("api_test/app_process")));
 
   // The app under test acts on URLs whose host is "localhost",
   // so the URLs we navigate to must have host "localhost".
@@ skipping 125 matching lines @@
 #else
 #define MAYBE_ExtensionProcessBalancing ExtensionProcessBalancing
 #endif
 // Test to verify that the policy of maximum share of extension processes is
 // properly enforced.
 IN_PROC_BROWSER_TEST_F(ProcessManagementTest, MAYBE_ExtensionProcessBalancing) {
   // Set max renderers to 6 so we can expect 2 extension processes to be
   // allocated.
   content::RenderProcessHost::SetMaxRendererProcessCount(6);
 
-  host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(embedded_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 = embedded_test_server()->GetURL(
       "/extensions/");
   GURL::Replacements replace_host;
   replace_host.SetHostStr("localhost");
   base_url = base_url.ReplaceComponents(replace_host);
 
@@ skipping 28 matching lines @@
   // background page, and one isolated app. We expect only 2 unique processes
   // hosting those extensions.
   extensions::ProcessMap* process_map = extensions::ProcessMap::Get(profile);
 
   EXPECT_GE((size_t) 6, process_map->size());
   EXPECT_EQ((size_t) 2, process_ids.size());
 }
 
 IN_PROC_BROWSER_TEST_F(ProcessManagementTest,
                        NavigateExtensionTabToWebViaPost) {
-  host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(embedded_test_server()->Start());
 
   // Load an extension.
   const extensions::Extension* extension = LoadExtension(
       test_data_dir_.AppendASCII("api_test/browser_action/popup_with_form"));
   ASSERT_TRUE(extension);
 
   // Navigate a tab to an extension page.
   GURL extension_url = extension->GetResourceURL("popup.html");
   ui_test_utils::NavigateToURL(browser(), extension_url);
@@ skipping 92 matching lines @@
       web_contents->GetMainFrame()->GetProcess();
   EXPECT_TRUE(extensions::ProcessMap::Get(profile())->Contains(
       extensions::kWebStoreAppId, new_process_host->GetID()));
 
   // Verify that Chrome Web Store is isolated in a separate renderer process.
   EXPECT_NE(old_process_host, new_process_host);
 }
 
 // This test verifies that blocked navigations to extensions pages do not
 // overwrite process-per-site map inside content/.
-IN_PROC_BROWSER_TEST_F(ExtensionBrowserTest,
+IN_PROC_BROWSER_TEST_F(ProcessManagementTest,
                        NavigateToBlockedExtensionPageInNewTab) {
-  host_resolver()->AddRule("*", "127.0.0.1");
   ASSERT_TRUE(embedded_test_server()->Start());
 
   // Load an extension, which will block a request for a specific page in it.
   const extensions::Extension* extension = LoadExtension(
       test_data_dir_.AppendASCII("web_request_site_process_registration"));
   ASSERT_TRUE(extension);
 
   WebContents* web_contents =
       browser()->tab_strip_model()->GetActiveWebContents();
   GURL blocked_url(extension->GetResourceURL("/blocked.html"));
@@ skipping 39 matching lines @@
   // the process is the same for all SiteInstances. This allows us to verify
   // that the site-to-process map for the extension hasn't been overwritten by
   // the process of the |blocked_url|.
   scoped_refptr<content::SiteInstance> new_site_instance =
       content::SiteInstance::CreateForURL(web_contents->GetBrowserContext(),
                                           extension->GetResourceURL(""));
   EXPECT_TRUE(new_site_instance->HasProcess());
   EXPECT_EQ(new_site_instance->GetProcess(),
             web_contents->GetSiteInstance()->GetProcess());
 }