PlzNavigate: implement process reuse for ServiceWorkers

content/browser/frame_host/navigation_handle_impl.h

Index: content/browser/frame_host/navigation_handle_impl.h
diff --git a/content/browser/frame_host/navigation_handle_impl.h b/content/browser/frame_host/navigation_handle_impl.h
index 5d716dfe5019ed913907d2c96915337229f86689..ba61a68d12d6368c38d50b0ee98289086921153d 100644
--- a/content/browser/frame_host/navigation_handle_impl.h
+++ b/content/browser/frame_host/navigation_handle_impl.h
@@ -380,6 +380,12 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
     source_location_ = source_location;
   }
 
+  // PlzNavigate
+  // Sets ID of the RenderProcessHost we expect the navigation to commit in.
+  // This is used to inform the RenderProcessHost to expect a navigation to the
+  // url we're navigating to.
+  void SetSpeculativeProcess(RenderProcessHost* speculative_host);
+
  private:
   friend class NavigationHandleImplTest;
 
@@ -426,6 +432,12 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
   // WillStartRequest and WillRedirectRequest to prevent the navigation.
   bool IsSelfReferentialURL();
 
+  // Updates the SiteURL this navigation is navigating to. This is called on
+  // redirects.
+  // PlzNavigate: When redirected cross-site, the speculative RenderProcessHost
+  // will stop expecting this navigation to commit.
+  void UpdateSiteURL();
+
   // See NavigationHandle for a description of those member variables.
   GURL url_;
   scoped_refptr<SiteInstance> starting_site_instance_;
@@ -561,6 +573,14 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
   // navigations initiated by Javascript.
   SourceLocation source_location_;
 
+  // The SiteURL of this navigation.

[Charlie Reis, 2017/05/18 00:58:34]

nit: The site URL of this navigation, as obtained from SiteInstance::GetSiteURL.

Maybe this is worth putting higher up, near something like original_url_?

[clamy, 2017/05/22 16:59:52]

Done.

+  GURL site_url_;
+
+  // PlzNavigate
+  // Used to inform a RenderProcessHost that we expect this navigation to commit
+  // in it.
+  int speculative_render_process_host_id_;
+
   base::WeakPtrFactory<NavigationHandleImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl);