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

chrome/browser/extensions/extension_process_manager.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_PROCESS_MANAGER_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_PROCESS_MANAGER_H_
 #pragma once
 
 #include <map>
 #include <set>
@@ skipping 59 matching lines @@
   virtual SiteInstance* GetSiteInstanceForURL(const GURL& url);
 
   // Registers a SiteInstance with |site_instance_id| as hosting the extension
   // with |extension_id|.
   void RegisterExtensionSiteInstance(int site_instance_id,
                                      const std::string& extension_id);
 
   // Unregisters the extension associated with |site_instance_id|.
   void UnregisterExtensionSiteInstance(int site_instance_id);
 
+  // Registers a RenderProcessHost with |host_id| as having permission to
+  // access extension bindings.
+  void RegisterProcessHostWithBindings(int host_id);

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

Can we just call these methods "RegisterProcess", "UnregisterProcess", etc?

Also, we will want to eventually keep track of the actual Extension. Because
that way we can lock down processes to only accessing the apis that some
extension in their process has.

Have you thought at all about whether your design will grow to handle that
easily? I'm not sure if we'd store the Extension* or an extension ID... Or about
how we'd keep them in sync.

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

Done
At the point where the process is registered, we have the Extension object, so
basically we can keep track of the extension that is running in this process as
well.

For the IsSuitableHost call, we'd need to derive the extension from the URL, but
that's possible

+
+  // Unregisters the RenderProcessHost with |host_id|.
+  void UnregisterProcessHostWithBindings(int host_id);
+
+  // True if this process host is known to have access to extension bindings.
+  bool IsProcessHostWithBindings(int host_id) const;
+
   // Returns the extension process that |url| is associated with if it exists.
   // This is not valid for hosted apps without the background permission, since
   // such apps may have multiple processes.
   virtual RenderProcessHost* GetExtensionProcess(const GURL& url);
 
   // Returns the process that the extension with the given ID is running in.
   RenderProcessHost* GetExtensionProcess(const std::string& extension_id);
 
   // Returns the Extension associated with the given SiteInstance id, if any.
   virtual const Extension* GetExtensionForSiteInstance(int site_instance_id);
@@ skipping 29 matching lines @@
   ExtensionHostSet background_hosts_;
 
   // The BrowsingInstance shared by all extensions in this profile.  This
   // controls process grouping.
   scoped_refptr<BrowsingInstance> browsing_instance_;
 
   // A map of site instance ID to the ID of the extension it hosts.
   typedef std::map<int, std::string> SiteInstanceIDMap;
   SiteInstanceIDMap extension_ids_;
 
+  // A set of render process host IDs that have access to extension bindings.
+  std::set<int> process_ids_;
+
   DISALLOW_COPY_AND_ASSIGN(ExtensionProcessManager);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_PROCESS_MANAGER_H_