PlzNavigate: Create the speculative renderer earlier.

content/browser/frame_host/frame_tree_node.cc

 // Copyright 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 "content/browser/frame_host/frame_tree_node.h"
 
 #include <queue>
 
 #include "base/command_line.h"
 #include "base/profiler/scoped_tracker.h"
@@ skipping 201 matching lines @@
   // the UI. Blink doesn't send throb notifications for JavaScript URLs, so it
   // is not done here either.
   if (!navigation_request->common_params().url.SchemeIs(
           url::kJavaScriptScheme)) {
     // TODO(fdegans): Check if this is a same-document navigation and set the
     // proper argument.
     DidStartLoading(true);
   }
 
   navigation_request_ = navigation_request.Pass();
+
+  render_manager()->BeginNavigation(*navigation_request_);

[clamy, 2015/06/09 16:01:29]

Please rename the function in RFHM DidSetNavigationRequest (or
DidCreateNavigationRequest) so that there is no confusion with
OnBeginNavigation.

[carlosk, 2015/06/10 11:53:08]

Done.

 }
 
 void FrameTreeNode::ResetNavigationRequest(bool is_commit) {
   CHECK(base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableBrowserSideNavigation));
   if (!navigation_request_)
     return;
   navigation_request_.reset();
 
   // During commit, the clean up of a speculative RenderFrameHost is done in
@@ skipping 68 matching lines @@
       FROM_HERE_WITH_EXPLICIT_FUNCTION(
           "465796 FrameTreeNode::DidStopLoading::End"));
 }
 
 void FrameTreeNode::DidChangeLoadProgress(double load_progress) {
   loading_progress_ = load_progress;
   frame_tree_->UpdateLoadProgress();
 }
 
 }  // namespace content