PlzNavigate: implement process reuse for ServiceWorkers

content/browser/frame_host/navigation_request.cc

 // Copyright 2014 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/navigation_request.h"
 
 #include <utility>
 
 #include "base/memory/ptr_util.h"
 #include "content/browser/appcache/appcache_navigation_handle.h"
@@ skipping 697 matching lines @@
   // Use the SiteInstance of the navigating RenderFrameHost to get access to
   // the StoragePartition. Using the url of the navigation will result in a
   // wrong StoragePartition being picked when a WebView is navigating.
   DCHECK_NE(AssociatedSiteInstanceType::NONE, associated_site_instance_type_);
   RenderFrameHostImpl* navigating_frame_host =
       associated_site_instance_type_ == AssociatedSiteInstanceType::SPECULATIVE
           ? frame_tree_node_->render_manager()->speculative_frame_host()
           : frame_tree_node_->current_frame_host();
   DCHECK(navigating_frame_host);
 
+  // Inform the NavigationHandle of the RenderProcessHost ID of the
+  // RenderProcessHost of the |navigating_frame_host|.
+  navigation_handle_->SetSpeculativeProcessID(
+      navigating_frame_host->GetProcess()->GetID(), !may_transfer_);

[Charlie Reis, 2017/05/15 03:41:52]

Yes, let's revisit https://codereview.chromium.org/2768813002/ or even land a
separate CL to rename may_transfer_, if you prefer.  I think this would be
another case where it's a bit hard to map the concepts onto each other.

[clamy, 2017/05/16 14:50:45]

The parameter was removed, though we should revisit
https://codereview.chromium.org/2768813002/ with a rename.

+
   BrowserContext* browser_context =
       frame_tree_node_->navigator()->GetController()->GetBrowserContext();
   StoragePartition* partition = BrowserContext::GetStoragePartition(
       browser_context, navigating_frame_host->GetSiteInstance());
   DCHECK(partition);
 
   // Only initialize the ServiceWorkerNavigationHandle if it can be created for
   // this frame.
   bool can_create_service_worker =
       (frame_tree_node_->pending_sandbox_flags() &
@@ skipping 134 matching lines @@
   DCHECK_EQ(request_params_.has_user_gesture, begin_params_.has_user_gesture);
 
   render_frame_host->CommitNavigation(response_.get(), std::move(body_),
                                       std::move(handle_), common_params_,
                                       request_params_, is_view_source_);
 
   frame_tree_node_->ResetNavigationRequest(true, true);
 }
 
 }  // namespace content