Perform navigation policy check on UI thread for --site-per-process.

content/public/test/content_browser_test_utils.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.
 
 #include "content/public/test/content_browser_test_utils.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/path_service.h"
 #include "base/run_loop.h"
+#include "content/browser/frame_host/render_frame_host_impl.h"
 #include "content/public/browser/navigation_controller.h"
 #include "content/public/browser/notification_source.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/content_paths.h"
 #include "content/public/test/browser_test_utils.h"
 #include "content/public/test/test_navigation_observer.h"
 #include "content/public/test/test_utils.h"
 #include "content/shell/browser/shell.h"
 #include "content/shell/browser/shell_javascript_dialog_manager.h"
+#include "content/test/test_frame_navigation_observer.h"
 #include "net/base/filename_util.h"
 
 namespace content {
 
 base::FilePath GetTestFilePath(const char* dir, const char* file) {
   base::FilePath path;
   PathService::Get(DIR_TEST_DATA, &path);
   return path.Append(base::FilePath().AppendASCII(dir).Append(
       base::FilePath().AppendASCII(file)));
 }
@@ skipping 19 matching lines @@
   TestNavigationObserver same_tab_observer(window->web_contents(), 1);
 
   window->LoadDataWithBaseURL(url, data, base_url);
   same_tab_observer.Wait();
 }
 
 void NavigateToURL(Shell* window, const GURL& url) {
   NavigateToURLBlockUntilNavigationsComplete(window, url, 1);
 }
 
+void NavigateFrameToURL(RenderFrameHostImpl* rfh, const GURL& url) {
+  TestFrameNavigationObserver observer(rfh);
+  rfh->NavigateToURL(url);

[Charlie Reis, 2014/04/24 20:37:50]

Hmm, I'm not sure we want to use RFH::NavigateToURL, do we?  Seems like it would
be more representative to go through NavigationController::LoadURLWithParams, as
in
https://codereview.chromium.org/209153003/diff/1/content/browser/site_per_pro....
 Does that have problems?

Similarly, I'm uncertain if we want this method to take in a RFH or a
FrameTreeNode.  I don't think we would want people trying to navigate RFHs that
aren't the current RFH, for example.

[nasko, 2014/04/24 22:31:12]

Done.

+  observer.Wait();
+}
+
 void WaitForAppModalDialog(Shell* window) {
   ShellJavaScriptDialogManager* dialog_manager=
       static_cast<ShellJavaScriptDialogManager*>(
           window->GetJavaScriptDialogManager());
 
   scoped_refptr<MessageLoopRunner> runner = new MessageLoopRunner();
   dialog_manager->set_dialog_request_callback(runner->QuitClosure());
   runner->Run();
 }
 
@@ skipping 16 matching lines @@
 }
 
 void ShellAddedObserver::ShellCreated(Shell* shell) {
   DCHECK(!shell_);
   shell_ = shell;
   if (runner_.get())
     runner_->QuitClosure().Run();
 }
 
 }  // namespace content