Fix RenderProcessHostUnitTest.RendererProcessLimit to pass with --site-per-process

content/browser/renderer_host/render_process_host_unittest.cc

Index: content/browser/renderer_host/render_process_host_unittest.cc
diff --git a/content/browser/renderer_host/render_process_host_unittest.cc b/content/browser/renderer_host/render_process_host_unittest.cc
index ca497a397e5b52edbf29ce562602a51fe4d66f08..92bef4679d4a9890593fc381840813cb9205e8e4 100644
--- a/content/browser/renderer_host/render_process_host_unittest.cc
+++ b/content/browser/renderer_host/render_process_host_unittest.cc
@@ -4,7 +4,9 @@
 
 #include <limits>
 
+#include "base/command_line.h"
 #include "content/public/common/content_constants.h"
+#include "content/public/common/content_switches.h"
 #include "content/public/test/mock_render_process_host.h"
 #include "content/test/test_render_view_host.h"
 
@@ -31,6 +33,12 @@ TEST_F(RenderProcessHostUnitTest, GuestsAreNotSuitableHosts) {
 
 #if !defined(OS_ANDROID)
 TEST_F(RenderProcessHostUnitTest, RendererProcessLimit) {
+  // This test shouldn't run with --site-per-process mode, since it doesn't
+  // allow renderer process reuse, which this test explicitly exercises.
+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(

[ppi, 2014/09/25 08:36:59]

The condition that disables the process reuse in RenderProcessHost is actually:

  if (command_line.HasSwitch(switches::kEnableStrictSiteIsolation) ||
      command_line.HasSwitch(switches::kSitePerProcess))

- would it make sense to match it here (i.e. disable for
kEnableStrictSiteIsolation as well) ?

+          switches::kSitePerProcess))
+    return;
+
   // Disable any overrides.
   RenderProcessHostImpl::SetMaxRendererProcessCount(0);