PlzNavigate: Create the speculative renderer earlier.

content/browser/security_exploit_browsertest.cc

 // Copyright (c) 2013 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 "base/command_line.h"
 #include "base/containers/hash_tables.h"
 #include "base/strings/utf_string_conversions.h"
 #include "content/browser/dom_storage/dom_storage_context_wrapper.h"
 #include "content/browser/dom_storage/session_storage_namespace_impl.h"
 #include "content/browser/frame_host/navigator.h"
@@ skipping 55 matching lines @@
             shell->web_contents()->GetRenderViewHost()->GetRoutingID());
 
   // Now, simulate a link click coming from the renderer.
   GURL extension_url("https://bar.com/simple_page.html");
   WebContentsImpl* wc = static_cast<WebContentsImpl*>(shell->web_contents());
   wc->GetFrameTree()->root()->navigator()->RequestOpenURL(
       wc->GetFrameTree()->root()->current_frame_host(), extension_url, nullptr,
       Referrer(), CURRENT_TAB, false, true);
 
   // Since the navigation above requires a cross-process swap, there will be a
-  // pending RenderViewHost. Ensure it exists and is in a different process
-  // than the initial page.
-  RenderViewHostImpl* pending_rvh =
-      wc->GetRenderManagerForTesting()->pending_render_view_host();
+  // speculative/pending RenderViewHost. Ensure it exists and is in a different
+  // process than the initial page.
+  RenderViewHostImpl* pending_rvh = nullptr;
+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnableBrowserSideNavigation)) {
+    RenderFrameHostImpl* speculative_rfh =
+        wc->GetRenderManagerForTesting()->speculative_frame_host_for_testing();
+    if (speculative_rfh)
+      pending_rvh = speculative_rfh->render_view_host();
+  } else {
+    pending_rvh = wc->GetRenderManagerForTesting()->pending_render_view_host();

[carlosk, 2015/06/08 18:32:43]

I didn't create an equivalent RFHM::speculative_render_view_host method there's
a TODO in this one stating it should be removed in the future.

+  }
+
   EXPECT_TRUE(pending_rvh != NULL);
   EXPECT_NE(shell->web_contents()->GetRenderViewHost()->GetProcess()->GetID(),
             pending_rvh->GetProcess()->GetID());
 
   return pending_rvh;
 }
 
 }  // namespace
 
 
@@ skipping 218 matching lines @@
   // "evil" message doesn't arrive in the intervening period.
   ASSERT_TRUE(content::ExecuteScript(
       interstitial_page->GetMainFrame(),
       "window.domAutomationController.send(\"okay2\");"));
   ASSERT_TRUE(message_queue.WaitForMessage(&message));
   ASSERT_EQ("\"okay2\"", message);
   ASSERT_EQ("\"okay2\"", interstitial->last_command());
 }
 
 }  // namespace content