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

content/browser/content_browser_client.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 CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #define CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_
 #pragma once
 
 #include <string>
 
 #include "base/callback_old.h"
 #include "content/common/content_client.h"
 #include "content/common/window_container_type.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebNotificationPresenter.h"
 
 class AccessTokenStore;
 class BrowserRenderProcessHost;
 class BrowserURLHandler;
 class CommandLine;
 class DevToolsManager;
 class FilePath;
 class GURL;
 class MHTMLGenerationManager;
 class PluginProcessHost;
 class QuotaPermissionContext;
+class RenderProcessHost;
 class RenderViewHost;
 class ResourceDispatcherHost;
 class SSLCertErrorHandler;
 class SSLClientAuthHandler;
 class SkBitmap;
 class TabContents;
 class TabContentsView;
 class WorkerProcessHost;
 struct DesktopNotificationHostMsg_Show_Params;
 struct MainFunctionParams;
@@ skipping 81 matching lines @@
 
   // Returns whether all instances of the specified effective URL should be
   // rendered by the same process, rather than using process-per-site-instance.
   virtual bool ShouldUseProcessPerSite(BrowserContext* browser_context,
                                        const GURL& effective_url) = 0;
 
   // Returns whether a specified URL is to be considered the same as any
   // SiteInstance.
   virtual bool IsURLSameAsAnySiteInstance(const GURL& url) = 0;
 
+  // Returns whether a specified URL should be rendered in a process with
+  // special privileges, e.g. special bindings.

[jam, 2011/09/26 15:46:16]

nit: it's unclear to me reading this what special privileges means?

[jochen (gone - plz use gerrit), 2011/09/26 15:56:04]

I tried to avoid saying extension bindings, since content shouldn't know about
extension bindings.

any suggestions how to improve this comment?

+  virtual bool ShouldRunInPrivilegedProcess(const GURL& url) = 0;
+
+  // Returns whether a new view for a given |site_url| can be launched in a
+  // given |process_host|.
+  virtual bool IsSuitableHost(RenderProcessHost* process_host,
+                              const GURL& site_url) = 0;
+
   // See CharacterEncoding's comment.
   virtual std::string GetCanonicalEncodingNameByAliasName(
       const std::string& alias_name) = 0;
 
   // Allows the embedder to pass extra command line flags.
   // switches::kProcessType will already be set at this point.
   virtual void AppendExtraCommandLineSwitches(CommandLine* command_line,
                                               int child_process_id) = 0;
 
   // Returns the locale used by the application.
@@ skipping 182 matching lines @@
   // This is called on a worker thread.
   virtual
       crypto::CryptoModuleBlockingPasswordDelegate* GetCryptoPasswordDelegate(
           const GURL& url) = 0;
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_CONTENT_BROWSER_CLIENT_H_