PlzNavigate: Make ServiceWorker work with PlzNavigate.

content/renderer/render_frame_impl.cc

 // 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.
 
 #include "content/renderer/render_frame_impl.h"
 
 #include <map>
 #include <string>
 
 #include "base/auto_reset.h"
@@ skipping 2558 matching lines @@
         document_state->set_load_type(DocumentState::LINK_LOAD_CACHE_STALE_OK);
         break;
       case WebURLRequest::ReturnCacheDataDontLoad:  // Don't re-post.
         document_state->set_load_type(DocumentState::LINK_LOAD_CACHE_ONLY);
         break;
       default:
         NOTREACHED();
     }
   }
 
-  // Create the serviceworker's per-document network observing object if it
-  // does not exist (When navigation happens within a page, the provider already
-  // exists).
-  if (!ServiceWorkerNetworkProvider::FromDocumentState(
+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kEnableBrowserSideNavigation)) {
+    // PlzNavigate
+    // Create a new ServiceWorker's network observing object for every
+    // navigation.
+    if (!content_initiated) {
+      ServiceWorkerProviderType provider_type =

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

There's a lot of repeated code here. How about:
bool browser_side_navigation = base::CommandLine...
if (browser_side_navigation && !content_initiated)
  return;
int service_worker_provider_id = browser_side_navigation ? ... : ...;
....

I think that would be clearer.

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

I reworked the whole part here.

+          SERVICE_WORKER_PROVIDER_FOR_WINDOW;
+      if ((frame->effectiveSandboxFlags() & blink::WebSandboxFlags::Origin) ==
+          blink::WebSandboxFlags::Origin) {
+        provider_type = SERVICE_WORKER_PROVIDER_FOR_SANDBOXED_FRAME;

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

Are the effectiveSandboxFlags() sent as part of the FrameHostMsg_BeginNavigation
message? I don't thing they are. In this case we need to skip the serviceworker.
There's no need to create a providerhost in the browser process. (see related
comments in SWNPH.c)

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

That's my mistake. I'm setting the provider_id to -1 in the browser for
sandboxed frames.

+      }
+
+      NavigationStateImpl* navigation_state =
+          static_cast<NavigationStateImpl*>(
+              DocumentState::FromDataSource(datasource)->navigation_state());
+      int service_worker_provider_id =
+          navigation_state->request_params().service_worker_provider_id;
+      scoped_ptr<ServiceWorkerNetworkProvider> network_provider(
+          new ServiceWorkerNetworkProvider(routing_id_, provider_type,
+                                           service_worker_provider_id));
+
+      ServiceWorkerNetworkProvider::AttachToDocumentState(
+          DocumentState::FromDataSource(datasource),
+          network_provider.Pass());
+    }
+  } else {
+    // Create the ServiceWorker's per-document network observing object if it
+    // does not exist (When navigation happens within a page, the provider
+    // already exists).
+    if (!ServiceWorkerNetworkProvider::FromDocumentState(
           DocumentState::FromDataSource(datasource))) {
-    ServiceWorkerProviderType provider_type =
-        SERVICE_WORKER_PROVIDER_FOR_WINDOW;
-    if ((frame->effectiveSandboxFlags() & blink::WebSandboxFlags::Origin) ==
-        blink::WebSandboxFlags::Origin) {
-      provider_type = SERVICE_WORKER_PROVIDER_FOR_SANDBOXED_FRAME;
+      ServiceWorkerProviderType provider_type =
+          SERVICE_WORKER_PROVIDER_FOR_WINDOW;
+      if ((frame->effectiveSandboxFlags() & blink::WebSandboxFlags::Origin) ==
+          blink::WebSandboxFlags::Origin) {
+        provider_type = SERVICE_WORKER_PROVIDER_FOR_SANDBOXED_FRAME;
+      }
+
+      scoped_ptr<ServiceWorkerNetworkProvider> network_provider(
+          new ServiceWorkerNetworkProvider(routing_id_, provider_type,
+                                           kInvalidServiceWorkerProviderId));
+
+      ServiceWorkerNetworkProvider::AttachToDocumentState(
+          DocumentState::FromDataSource(datasource),
+          network_provider.Pass());
     }
-    scoped_ptr<ServiceWorkerNetworkProvider> network_provider(
-        new ServiceWorkerNetworkProvider(routing_id_, provider_type));
-    ServiceWorkerNetworkProvider::AttachToDocumentState(
-        DocumentState::FromDataSource(datasource),
-        network_provider.Pass());
   }
 }
 
 void RenderFrameImpl::didStartProvisionalLoad(blink::WebLocalFrame* frame,
                                               double triggering_event_time) {
   DCHECK(!frame_ || frame_ == frame);
   WebDataSource* ds = frame->provisionalDataSource();
 
   // In fast/loader/stop-provisional-loads.html, we abort the load before this
   // callback is invoked.
@@ skipping 2291 matching lines @@
   WebDataSource* provisional_data_source = frame_->provisionalDataSource();
   WebDataSource* current_data_source = frame_->dataSource();
   WebDataSource* data_source =
       provisional_data_source ? provisional_data_source : current_data_source;
 
   // The current entry can only be replaced if there already is an entry in the
   // history list.
   if (data_source && render_view_->history_list_length_ > 0) {
     should_replace_current_entry = data_source->replacesCurrentHistoryItem();
   }
+
+  DCHECK_EQ(FETCH_REQUEST_MODE_SAME_ORIGIN,
+            GetFetchRequestModeForWebURLRequest(*request));
+  DCHECK_EQ(FETCH_CREDENTIALS_MODE_INCLUDE,
+            GetFetchCredentialsModeForWebURLRequest(*request));
+  DCHECK(GetFetchRedirectModeForWebURLRequest(*request) ==
+         FetchRedirectMode::MANUAL_MODE);
+  DCHECK_IMPLIES(!frame_->parent(),
+                 GetRequestContextFrameTypeForWebURLRequest(*request) ==
+                 REQUEST_CONTEXT_FRAME_TYPE_TOP_LEVEL);
+  DCHECK_IMPLIES(frame_->parent(),
+                 GetRequestContextFrameTypeForWebURLRequest(*request) ==
+                 REQUEST_CONTEXT_FRAME_TYPE_NESTED);
+
   Send(new FrameHostMsg_BeginNavigation(
       routing_id_,
       MakeCommonNavigationParams(request, should_replace_current_entry),
       BeginNavigationParams(
-          request->httpMethod().latin1(), GetWebURLRequestHeaders(*request),
-          GetLoadFlagsForWebURLRequest(*request), request->hasUserGesture()),
+          request->httpMethod().latin1(),
+          GetWebURLRequestHeaders(*request),
+          GetLoadFlagsForWebURLRequest(*request),
+          request->hasUserGesture(),
+          request->skipServiceWorker(),
+          GetRequestContextTypeForWebURLRequest(*request)),
       GetRequestBodyForWebURLRequest(*request)));
 }
 
 void RenderFrameImpl::LoadDataURL(const CommonNavigationParams& params,
                                   WebFrame* frame) {
   // A loadData request with a specified base URL.
   std::string mime_type, charset, data;
   if (net::DataURL::Parse(params.url, &mime_type, &charset, &data)) {
     const GURL base_url = params.base_url_for_data_url.is_empty() ?
         params.url : params.base_url_for_data_url;
@@ skipping 251 matching lines @@
   mojo::ServiceProviderPtr service_provider;
   mojo::URLRequestPtr request(mojo::URLRequest::New());
   request->url = mojo::String::From(url);
   mojo_shell_->ConnectToApplication(request.Pass(), GetProxy(&service_provider),
                                     nullptr, nullptr,
                                     base::Bind(&OnGotContentHandlerID));
   return service_provider.Pass();
 }
 
 }  // namespace content