Isolate hosted apps.

Hosted apps no longer need to be a special case that does not require
a dedicated process. This change removes the special case logic and
isolates all extensions and all hosted apps in their own processes.

Change-Id: Id6322038fcf5bef91775cec20ea61cf7a3f6b458
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1628334Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Aaron Colwell <acolwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#664018}

chrome/browser/extensions/chrome_content_browser_client_extensions_part.cc

@@ -411,20 +411,8 @@ bool ChromeContentBrowserClientExtensionsPart::DoesSiteRequireDedicatedProcess(
     const GURL& effective_site_url) {
   const Extension* extension = GetEnabledExtensions(browser_or_resource_context)
                                    ->GetExtensionOrAppByURL(effective_site_url);
-  if (!extension)
-    return false;
-
-  // Always isolate Chrome Web Store.
-  if (extension->id() == kWebStoreAppId)
-    return true;
-
-  // Extensions should be isolated, except for hosted apps. Isolating hosted
-  // apps is a good idea, but ought to be a separate knob.
-  if (extension->is_hosted_app())
-    return false;
-
   // Isolate all extensions.
-  return true;
+  return extension != nullptr;
 }
 
 // static