Remove command line/field trial support and configs for Isolate Extensions.

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 306 matching lines @@
   content::TestNavigationObserver nav_observer(web_contents, 1);
   EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
       web_contents, navigation_starting_script, &ignored_script_result));
 
   // Verify that the navigation succeeded.
   nav_observer.Wait();
   EXPECT_EQ(web_url, web_contents->GetLastCommittedURL());
 
   // Verify that the navigation transferred the contents to another renderer
   // process.
-  if (extensions::IsIsolateExtensionsEnabled()) {
-    content::RenderProcessHost* new_process_host =
-        web_contents->GetMainFrame()->GetProcess();
-    EXPECT_NE(old_process_host, new_process_host);
-  }
+  content::RenderProcessHost* new_process_host =
+      web_contents->GetMainFrame()->GetProcess();
+  EXPECT_NE(old_process_host, new_process_host);
 }
 
 IN_PROC_BROWSER_TEST_F(ChromeWebStoreProcessTest,
                        NavigateWebTabToChromeWebStoreViaPost) {
   // Navigate a tab to a web page with a form.
   GURL web_url = embedded_test_server()->GetURL("foo.com", "/form.html");
   ui_test_utils::NavigateToURL(browser(), web_url);
   WebContents* web_contents =
       browser()->tab_strip_model()->GetActiveWebContents();
   EXPECT_EQ(web_url, web_contents->GetLastCommittedURL());
@@ skipping 17 matching lines @@
       "setTimeout(\n"
       "    function() { window.domAutomationController.send(true); },\n"
       "    0);\n";
 
   // Trigger a renderer-initiated POST navigation (via the form) to a Chrome Web
   // Store gallery URL (which will commit into a chrome-extension://cws-app-id).
   bool ignored_script_result = false;
   content::TestNavigationObserver nav_observer(web_contents, 1);
   content::RenderProcessHostWatcher crash_observer(
       web_contents->GetMainFrame()->GetProcess(),
       content::RenderProcessHostWatcher::WATCH_FOR_PROCESS_EXIT);

[ncarter (slow), 2017/05/01 20:15:19]

Remove |crash_observer|

[nasko, 2017/05/01 21:25:37]

Done.

 
   EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
       web_contents, navigation_starting_script, &ignored_script_result));
 
-  // When --isolate-extensions is enabled, the expectation is that the store
-  // will be properly put in its own process, otherwise the renderer process
-  // is going to be terminated.
-  if (!extensions::IsIsolateExtensionsEnabled()) {
-    crash_observer.Wait();
-    return;
-  }
-
+  // The expectation is that the store will be properly put in its own process,
+  // otherwise the renderer process is going to be terminated.
   // Verify that the navigation succeeded.
   nav_observer.Wait();
   EXPECT_EQ(cws_web_url, web_contents->GetLastCommittedURL());
 
   // Verify that we really have the Chrome Web Store app loaded in the Web
   // Contents.
   content::RenderProcessHost* new_process_host =
       web_contents->GetMainFrame()->GetProcess();
   EXPECT_TRUE(extensions::ProcessMap::Get(profile())->Contains(
       extensions::kWebStoreAppId, new_process_host->GetID()));
@@ skipping 59 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());
 }