Adding a command line flag to specify renderer process limit

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"
 
 #if defined(OS_WIN)
@@ skipping 183 matching lines @@
     return request_context->GetURLRequestContext();
   }
 
  private:
   virtual ~RendererURLRequestContextSelector() {}
 
   scoped_refptr<net::URLRequestContextGetter> request_context_;
   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;
 
   // Defines the maximum number of renderer processes according to the
   // amount of installed memory as reported by the OS. The table
   // values are calculated by assuming that you want the renderers to
   // use half of the installed ram and assuming that each tab uses
   // ~40MB, however the curve is not linear but piecewise linear with
   // interleaved slopes of 3 and 2.
   // If you modify this table you need to adjust browser\browser_uitest.cc
@@ skipping 21 matching lines @@
   if (!max_count) {
     size_t memory_tier = base::SysInfo::AmountOfPhysicalMemoryMB() / 256;
     if (memory_tier >= arraysize(kMaxRenderersByRamTier))
       max_count = content::kMaxRendererProcessCount;
     else
       max_count = kMaxRenderersByRamTier[memory_tier];
   }
   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;
 }
 
 RenderProcessHostImpl::RenderProcessHostImpl(
     content::BrowserContext* browser_context)
         : fast_shutdown_started_(false),
           deleting_soon_(false),
           pending_views_(0),
           visible_widgets_(0),
           backgrounded_(true),
@@ skipping 1025 matching lines @@
 void RenderProcessHostImpl::OnRevealFolderInOS(const FilePath& path) {
   // Only honor the request if appropriate persmissions are granted.
   if (ChildProcessSecurityPolicy::GetInstance()->CanReadFile(GetID(), path))
     content::GetContentClient()->browser()->OpenItem(path);
 }
 
 void RenderProcessHostImpl::OnSavedPageAsMHTML(int job_id, int64 data_size) {
   content::GetContentClient()->browser()->GetMHTMLGenerationManager()->
       MHTMLGenerated(job_id, data_size);
 }