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

chrome/browser/extensions/extension_process_manager.h

Index: chrome/browser/extensions/extension_process_manager.h
diff --git a/chrome/browser/extensions/extension_process_manager.h b/chrome/browser/extensions/extension_process_manager.h
index cf1d94207dc2856ca9157c1218e2f3ce6346244d..4698dff7043112742658b89310894685739acc28 100644
--- a/chrome/browser/extensions/extension_process_manager.h
+++ b/chrome/browser/extensions/extension_process_manager.h
@@ -77,6 +77,16 @@ class ExtensionProcessManager : public NotificationObserver {
   // 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.
@@ -126,6 +136,9 @@ class ExtensionProcessManager : public NotificationObserver {
   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);
 };