PlzNavigate: Create the speculative renderer earlier.

content/browser/frame_host/render_frame_host_manager.h

 // 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.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_
 
 #include <list>
 
 #include "base/basictypes.h"
@@ skipping 366 matching lines @@
   // Deletes any proxy hosts associated with this node. Used during destruction
   // of WebContentsImpl.
   void ResetProxyHosts();
 
   // Returns the routing id for a RenderFrameHost or RenderFrameHostProxy
   // that has the given SiteInstance and is associated with this
   // RenderFrameHostManager. Returns MSG_ROUTING_NONE if none is found.
   int GetRoutingIdForSiteInstance(SiteInstance* site_instance);
 
   // PlzNavigate
-  // Notifies the RFHM that a navigation has begun so that it can speculatively
-  // create a new RenderFrameHost (and potentially a new process) if needed.
-  void BeginNavigation(const NavigationRequest& request);
+  // Notifies the RenderFrameHostManager that a new NavigationRequest has been
+  // set in the frame node so that it can speculatively create a new

[nasko, 2015/06/12 17:22:22]

nit: Let's use actual class names, so it is more concrete. s/frame
node/FrameTreeNode/.

[carlosk, 2015/06/15 19:11:54]

Done.

+  // RenderFrameHost (and potentially a new process) if needed.
+  void DidSetNavigationRequest(const NavigationRequest& request);
 
   // PlzNavigate
   // Called (possibly several times) during a navigation to select or create an
   // appropriate RenderFrameHost for the provided URL. The returned pointer will
   // be for the current or the speculative RenderFrameHost and the instance is
   // owned by this manager.
   RenderFrameHostImpl* GetFrameHostForNavigation(
       const NavigationRequest& request);
 
   // PlzNavigate
@@ skipping 334 matching lines @@
   bool should_reuse_web_ui_;
 
   base::WeakPtrFactory<RenderFrameHostManager> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_MANAGER_H_