PlzNavigate: implement process reuse for ServiceWorkers

content/browser/frame_host/render_frame_host_impl.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_IMPL_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 937 matching lines @@
                                  const base::string16& user_input);
 
   // Returns ownership of the NavigationHandle associated with a navigation that
   // just committed.
   std::unique_ptr<NavigationHandleImpl> TakeNavigationHandleForCommit(
       const FrameHostMsg_DidCommitProvisionalLoad_Params& params);
 
   // Called by |beforeunload_timeout_| when the beforeunload timeout fires.
   void BeforeUnloadTimeout();
 
+  // Called when a navigation commits succesfully to |url|. This will update
+  // |last_committed_site_url_| if it's not equal to the site url corresponding
+  // to |url|.
+  void SetLastCommittedSiteUrl(const GURL& url);
+
   // For now, RenderFrameHosts indirectly keep RenderViewHosts alive via a
   // refcount that calls Shutdown when it reaches zero.  This allows each
   // RenderFrameHostManager to just care about RenderFrameHosts, while ensuring
   // we have a RenderViewHost for each RenderFrameHost.
   // TODO(creis): RenderViewHost will eventually go away and be replaced with
   // some form of page context.
   RenderViewHostImpl* const render_view_host_;
 
   RenderFrameHostDelegate* const delegate_;
 
@@ skipping 23 matching lines @@
   // may change its current RenderFrameHost while this child is pending
   // deletion, and GetParent() should never return a different value.
   RenderFrameHostImpl* parent_;
 
   // Track this frame's last committed URL.
   GURL last_committed_url_;
 
   // Track this frame's last committed origin.
   url::Origin last_committed_origin_;
 
+  // Track the site URL of the last site we committed succesfully.

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

nit: successfully
(Maybe also mention: as obtained from SiteInstance::GetSiteURL)

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

Done.

+  GURL last_committed_site_url_;
+
   // The most recent non-error URL to commit in this frame.  Remove this in
   // favor of GetLastCommittedURL() once PlzNavigate is enabled or cross-process
   // transfers work for net errors.  See https://crbug.com/588314.
   GURL last_successful_url_;
 
   // The mapping of pending JavaScript calls created by
   // ExecuteJavaScript and their corresponding callbacks.
   std::map<int, JavaScriptResultCallback> javascript_callbacks_;
   std::map<uint64_t, VisualStateCallback> visual_state_callbacks_;
 
@@ skipping 222 matching lines @@
 
   // NOTE: This must be the last member.
   base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_