Delegate decision what site instances can be rendered in what process to chrome

chrome/browser/renderer_host/chrome_render_view_host_observer.cc

 // Copyright (c) 2011 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 "chrome/browser/renderer_host/chrome_render_view_host_observer.h"
 
 #include "base/command_line.h"
 #include "chrome/browser/dom_operation_notification_details.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/net/predictor.h"
@@ skipping 76 matching lines @@
   }
 
   // Enable extension bindings for the renderer. Currently only extensions,
   // packaged apps, and hosted component apps use extension bindings.
   Extension::Type type = extension->GetType();
   if (type == Extension::TYPE_EXTENSION ||
       type == Extension::TYPE_USER_SCRIPT ||
       type == Extension::TYPE_PACKAGED_APP ||
       (type == Extension::TYPE_HOSTED_APP &&
        extension->location() == Extension::COMPONENT)) {
     render_view_host()->AllowBindings(BindingsPolicy::EXTENSION);

[Aaron Boodman, 2011/09/27 19:26:49]

I had hoped we'd be able to get rid of BindingsPolicy::EXTENSION and
ChildSecurityPolicy::GrantExtensionBindings().

Looking through the code, I'm not even sure if the RVH flag is used for anything
anymore. ExtensionFunctionDispatcher checks this in a more direct way (just by
looking at the URL).

And the process flag may be obsoleted by the stuff you just added to EPM.

[jochen (gone - plz use gerrit), 2011/09/27 20:41:40]

True, looks like we could get rid of this entirely. I'll do this in a follow-up
CL. ok?

[Aaron Boodman, 2011/09/27 20:50:53]

Actually, now that I look at this, how come we can't just reuse
RegisterExtensionSiteInstance?.

It looks like it gets called everywhere this gets called, except that it also
runs for themes. And I don't think that even matters because we actually enforce
bindings (in ExtensionFunctionDispatcher) differently.

     ChildProcessSecurityPolicy::GetInstance()->GrantExtensionBindings(
         render_view_host()->process()->id());
+    process_manager->RegisterProcessHostWithBindings(
+        render_view_host()->process()->id());
   }
 }
 
 void ChromeRenderViewHostObserver::InitRenderViewForExtensions() {
   const Extension* extension = GetExtension();
   if (!extension)
     return;
 
   SiteInstance* site_instance = render_view_host()->site_instance();
   Profile* profile = Profile::FromBrowserContext(
@@ skipping 48 matching lines @@
 }
 
 void ChromeRenderViewHostObserver::OnDomOperationResponse(
     const std::string& json_string, int automation_id) {
   DomOperationNotificationDetails details(json_string, automation_id);
   NotificationService::current()->Notify(
       chrome::NOTIFICATION_DOM_OPERATION_RESPONSE,
       Source<RenderViewHost>(render_view_host()),
       Details<DomOperationNotificationDetails>(&details));
 }