Move existing kSitePerProcess checks to a policy-oracle object

content/public/test/browser_test_utils.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_TEST_BROWSER_TEST_UTILS_H_
 #define CONTENT_PUBLIC_TEST_BROWSER_TEST_UTILS_H_
 
 #include <queue>
 #include <string>
 #include <vector>
 
 #include "base/callback.h"
+#include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/memory/ref_counted.h"
 #include "base/process/process.h"
 #include "base/strings/string16.h"
 #include "content/public/browser/browser_message_filter.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/render_process_host_observer.h"
 #include "content/public/browser/web_contents_observer.h"
@@ skipping 216 matching lines @@
 // Waits for an interstitial page to detach from given web contents.
 void WaitForInterstitialDetach(content::WebContents* web_contents);
 
 // Runs task and waits for an interstitial page to detach from given web
 // contents. Prefer this over WaitForInterstitialDetach if web_contents may be
 // destroyed by the time WaitForInterstitialDetach is called (e.g. when waiting
 // for an interstitial detach after closing a tab).
 void RunTaskAndWaitForInterstitialDetach(content::WebContents* web_contents,
                                          const base::Closure& task);
 
+// Returns true if all sites are isolated. Typically used to bail from a test
+// that is incompatible with --site-per-process.
+bool AreAllSitesIsolatedForTesting();
+
+// Appends --site-per-process to the command line, enabling tests to exercise
+// site isolation and cross-process iframes.
+//
+// TODO(nick): In some places this method is called from the top of a test
+// body. That's not strictly safe (it's setting a command line after it
+// already may have been read). We should try make that pattern safer, as it
+// makes browser tests easier to write.
+void IsolateAllSitesForTesting(base::CommandLine* command_line);
+
 // Waits until all resources have loaded in the given RenderFrameHost.
 // When the load completes, this function sends a "pageLoadComplete" message
 // via domAutomationController. The caller should make sure this extra
 // message is handled properly.
 bool WaitForRenderFrameReady(RenderFrameHost* rfh) WARN_UNUSED_RESULT;
 
 // Watches title changes on a WebContents, blocking until an expected title is
 // set.
 class TitleWatcher : public WebContentsObserver {
  public:
@@ skipping 191 matching lines @@
   int routing_id_;
   bool deleted_;
   scoped_ptr<base::RunLoop> runner_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameDeletedObserver);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_TEST_BROWSER_TEST_UTILS_H_