PlzNavigate: implement process reuse for ServiceWorkers

content/browser/frame_host/navigation_handle_impl.h

 // Copyright 2015 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_NAVIGATION_HANDLE_IMPL_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_
 
 #include "content/public/browser/navigation_handle.h"
 
 #include <stddef.h>
@@ skipping 362 matching lines @@
 
   CSPDisposition should_check_main_world_csp() const {
     return should_check_main_world_csp_;
   }
 
   const SourceLocation& source_location() const { return source_location_; }
   void set_source_location(const SourceLocation& source_location) {
     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;
 
   NavigationHandleImpl(const GURL& url,
                        const std::vector<GURL>& redirect_chain,
                        FrameTreeNode* frame_tree_node,
                        bool is_renderer_initiated,
                        bool is_same_page,
                        const base::TimeTicks& navigation_start,
                        int pending_nav_entry_id,
@@ skipping 26 matching lines @@
   State state() const { return state_; }
 
   // Populates |throttles_| with the throttles for this navigation.
   void RegisterNavigationThrottles();
 
   // Checks for attempts to navigate to a page that is already referenced more
   // than once in the frame's ancestors.  This is a helper function used by
   // 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_;
   Referrer sanitized_referrer_;
   bool has_user_gesture_;
   ui::PageTransition transition_;
   bool is_external_protocol_;
   net::Error net_error_code_;
   RenderFrameHostImpl* render_frame_host_;
   const bool is_renderer_initiated_;
@@ skipping 115 matching lines @@
   CSPDisposition should_check_main_world_csp_;
 
   // Whether or not the navigation results from the submission of a form.
   bool is_form_submission_;
 
   // PlzNavigate
   // Information about the JavaScript that started the navigation. For
   // 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);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATION_HANDLE_IMPL_H_