Add UMA stats for predicted process counts with out-of-process iframes.

content/browser/renderer_host/render_process_host_impl.cc

 // Copyright (c) 2012 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.
 
 // Represents the browser side of the browser <--> renderer communication
 // channel. There will be one RenderProcessHost per renderer process.
 
 #include "content/browser/renderer_host/render_process_host_impl.h"
 
 #include <algorithm>
@@ skipping 1502 matching lines @@
   if (!suitable_renderers.empty()) {
     int suitable_count = static_cast<int>(suitable_renderers.size());
     int random_index = base::RandInt(0, suitable_count - 1);
     return suitable_renderers[random_index];
   }
 
   return NULL;
 }
 
 // static
-bool RenderProcessHostImpl::ShouldUseProcessPerSite(
+bool RenderProcessHost::ShouldUseProcessPerSite(
     BrowserContext* browser_context,
     const GURL& url) {
   // Returns true if we should use the process-per-site model.  This will be
   // the case if the --process-per-site switch is specified, or in
   // process-per-site-instance for particular sites (e.g., WebUI).
 
   const CommandLine& command_line = *CommandLine::ForCurrentProcess();
   if (command_line.HasSwitch(switches::kProcessPerSite))
     return true;
 
   // We want to consolidate particular sites like WebUI when we are using
   // process-per-tab or process-per-site-instance models.
   // Note that --single-process is handled in ShouldTryToUseExistingProcessHost.
 
   if (GetContentClient()->browser()->
           ShouldUseProcessPerSite(browser_context, url)) {
     return true;
   }
 
   // DevTools pages have WebUI type but should not reuse the same host.
   if (WebUIControllerFactoryRegistry::GetInstance()->UseWebUIForURL(

[darin (slow to review), 2013/06/12 04:40:48]

By the way, perhaps it would be better to perform this test before asking the
ContentBrowserClient?  It seems like this second check could override what the
ContentBrowserClient indicates, which could be confusing.  Might as well not
even bother consulting ContentBrowserClient in the cases where we are going to
ignore what it has to say.

[Charlie Reis, 2013/06/12 18:59:16]

Good point.  I'll throw together a small separate CL to fix that.

           browser_context, url) &&
       !url.SchemeIs(chrome::kChromeDevToolsScheme)) {
     return true;
   }
 
   // In all other cases, don't use process-per-site logic.
   return false;
 }
 
 // static
@@ skipping 243 matching lines @@
       continue;
 
     RenderViewHost* rvh =
         RenderViewHost::From(const_cast<RenderWidgetHost*>(widget));
 
     rvh->UpdateWebkitPreferences(rvh->GetWebkitPreferences());
   }
 }
 
 }  // namespace content