Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1133)

Unified Diff: content/browser/renderer_host/render_process_host_impl.h

Issue 10575014: Move process-per-site logic from BrowsingInstance to RenderProcessHost. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Clean up RPHs in test. Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_process_host_impl.h
diff --git a/content/browser/renderer_host/render_process_host_impl.h b/content/browser/renderer_host/render_process_host_impl.h
index d16181fc16641ad89466a9564e9354496c09c283..20de0204a67063141baee669cd0d41d0bc3cb06a 100644
--- a/content/browser/renderer_host/render_process_host_impl.h
+++ b/content/browser/renderer_host/render_process_host_impl.h
@@ -52,6 +52,14 @@ class CONTENT_EXPORT RenderProcessHostImpl
public ChildProcessLauncher::Client,
public base::WaitableEventWatcher::Delegate {
public:
+ // Map of site to process, to ensure we only have one RenderProcessHost per
+ // site in process-per-site mode.
+ typedef base::hash_map<std::string, RenderProcessHost*> SiteProcessMap;
+
+ // Map of BrowserContext to SiteProcessMap, for use in process-per-site mode.
+ typedef base::hash_map<content::BrowserContext*, SiteProcessMap>
+ ContextSiteProcessMap;
+
RenderProcessHostImpl(BrowserContext* browser_context, bool is_guest);
virtual ~RenderProcessHostImpl();
@@ -124,7 +132,7 @@ class CONTENT_EXPORT RenderProcessHostImpl
// Register/unregister the host identified by the host id in the global host
// list.
static void RegisterHost(int host_id, RenderProcessHost* host);
- static void UnregisterHost(int host_id);
+ static void UnregisterHost(int host_id, RenderProcessHost* host);
// Returns true if the given host is suitable for launching a new view
// associated with the given browser context.

Powered by Google App Engine
This is Rietveld 408576698