PlzNavigate: Make ServiceWorker work with PlzNavigate.

content/browser/frame_host/navigation_handle_impl.cc

 // 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.
 
 #include "content/browser/frame_host/navigation_handle_impl.h"
 
+#include "base/command_line.h"
 #include "content/browser/frame_host/navigator_delegate.h"
+#include "content/browser/service_worker/service_worker_context_wrapper.h"
+#include "content/public/browser/browser_context.h"
+#include "content/public/browser/browser_thread.h"
 #include "content/public/browser/content_browser_client.h"
+#include "content/public/browser/storage_partition.h"
 #include "content/public/common/content_client.h"
+#include "content/public/common/content_switches.h"
 #include "net/url_request/redirect_info.h"
 
 namespace content {
 
 // static
 scoped_ptr<NavigationHandleImpl> NavigationHandleImpl::Create(
     const GURL& url,
     bool is_main_frame,
-    NavigatorDelegate* delegate) {
-  return scoped_ptr<NavigationHandleImpl>(
-      new NavigationHandleImpl(url, is_main_frame, delegate));
+    NavigatorDelegate* delegate,
+    int service_worker_provider_id) {
+  return scoped_ptr<NavigationHandleImpl>(new NavigationHandleImpl(
+      url, is_main_frame, delegate, service_worker_provider_id));
 }
 
-NavigationHandleImpl::NavigationHandleImpl(const GURL& url,
-                                           const bool is_main_frame,
-                                           NavigatorDelegate* delegate)
+
+NavigationHandleImpl::NavigationHandleImpl(
+    const GURL& url,
+    bool is_main_frame,
+    NavigatorDelegate* delegate,
+    int service_worker_provider_id)
     : url_(url),
       is_main_frame_(is_main_frame),
       is_post_(false),
       has_user_gesture_(false),
       transition_(ui::PAGE_TRANSITION_LINK),
       is_external_protocol_(false),
       net_error_code_(net::OK),
       render_frame_host_(nullptr),
       is_same_page_(false),
       state_(INITIAL),
       is_transferring_(false),
-      delegate_(delegate) {
+      delegate_(delegate),
+      service_worker_provider_id_(service_worker_provider_id) {
+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnableBrowserSideNavigation)) {
+    BrowserContext* browser_context = delegate_->GetBrowserContext();
+    DCHECK(browser_context);
+    StoragePartition* partition = BrowserContext::GetStoragePartitionForSite(

[michaeln, 2015/10/01 01:42:15]

picking the partition based on the url is actually not correct, there's a bug
open related to this. maybe put a comment referring to crbug/513539

[Fabrice (no longer in Chrome), 2015/10/01 18:29:54]

Done.

+        browser_context, url);
+    DCHECK(partition);
+    service_worker_context_ = scoped_refptr<ServiceWorkerContextWrapper>(
+        static_cast<ServiceWorkerContextWrapper*>(
+            partition->GetServiceWorkerContext()));
+  }
   delegate_->DidStartNavigation(this);
 }
 
 NavigationHandleImpl::~NavigationHandleImpl() {
+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnableBrowserSideNavigation)) {
+    // PlzNavigate

[clamy, 2015/10/01 12:41:26]

nit: I would put this comment above the if condition (same in other places where
this happens).
Also precise what type of Host we're talking about by using the full class name.

[Fabrice (no longer in Chrome), 2015/10/01 18:29:54]

Done.

+    // Post task to IO thread to destroy the Host.
+    BrowserThread::PostTask(
+        BrowserThread::IO,
+        FROM_HERE,
+        base::Bind(&ServiceWorkerContextWrapper::RemoveNavigationProviderHost,
+                   service_worker_context_,
+                   service_worker_provider_id_));
+  }
   delegate_->DidFinishNavigation(this);
 }
 
 const GURL& NavigationHandleImpl::GetURL() {
   return url_;
 }
 
 bool NavigationHandleImpl::IsInMainFrame() {
   return is_main_frame_;
 }
@@ skipping 155 matching lines @@
 void NavigationHandleImpl::DidCommitNavigation(
     bool same_page,
     RenderFrameHostImpl* render_frame_host) {
   CHECK_IMPLIES(render_frame_host_, render_frame_host_ == render_frame_host);
   is_same_page_ = same_page;
   render_frame_host_ = render_frame_host;
   state_ = net_error_code_ == net::OK ? DID_COMMIT : DID_COMMIT_ERROR_PAGE;
 }
 
 }  // namespace content