Adding a command line flag to specify renderer process limit

content/browser/renderer_host/render_process_host_impl.cc

Index: content/browser/renderer_host/render_process_host_impl.cc
diff --git a/content/browser/renderer_host/render_process_host_impl.cc b/content/browser/renderer_host/render_process_host_impl.cc
index 657c8bb7f94ab668879cfa24ea4e67e5378e33f2..59ebcd001de51669e895136ed6239a8e9a704870 100644
--- a/content/browser/renderer_host/render_process_host_impl.cc
+++ b/content/browser/renderer_host/render_process_host_impl.cc
@@ -201,9 +201,16 @@ class RendererURLRequestContextSelector
   scoped_refptr<net::URLRequestContextGetter> media_request_context_;
 };
 
-size_t max_renderer_count_override = 0;
+// the global list of all renderer processes
+base::LazyInstance<IDMap<content::RenderProcessHost> >::Leaky
+    g_all_hosts = LAZY_INSTANCE_INITIALIZER;
 
-size_t GetMaxRendererProcessCount() {
+}  // namespace
+
+size_t max_renderer_count_override;

[Charlie Reis, 2012/02/03 23:05:34]

This should probably become a static on RenderProcessHost, now that it's not in
a local namespace.

[nasko, 2012/02/07 00:42:05]

Done.

+
+// static
+size_t content::RenderProcessHost::GetMaxRendererProcessCount() {
   if (max_renderer_count_override)
     return max_renderer_count_override;
 
@@ -245,18 +252,11 @@ size_t GetMaxRendererProcessCount() {
   return max_count;
 }
 
-// the global list of all renderer processes
-base::LazyInstance<IDMap<content::RenderProcessHost> >::Leaky
-    g_all_hosts = LAZY_INSTANCE_INITIALIZER;
-
-}  // namespace
-
 // static
 bool g_run_renderer_in_process_ = false;
 
 // static
-void content::RenderProcessHost::SetMaxRendererProcessCountForTest(
-    size_t count) {
+void content::RenderProcessHost::SetMaxRendererProcessCount(size_t count) {
   max_renderer_count_override = count;
 }