[Proof-of-concept] PlzNavigate and Service Worker

content/browser/service_worker/service_worker_request_handler.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/service_worker/service_worker_request_handler.h"
 
 #include <string>
 
+#include "base/command_line.h"
+#include "content/browser/loader/navigation_url_loader_impl_core.h"
 #include "content/browser/service_worker/service_worker_context_core.h"
 #include "content/browser/service_worker/service_worker_context_wrapper.h"
 #include "content/browser/service_worker/service_worker_provider_host.h"
 #include "content/browser/service_worker/service_worker_registration.h"
 #include "content/browser/service_worker/service_worker_url_request_job.h"
 #include "content/browser/service_worker/service_worker_utils.h"
 #include "content/common/resource_request_body.h"
 #include "content/common/service_worker/service_worker_types.h"
 #include "content/public/browser/resource_context.h"
+#include "content/public/common/content_switches.h"
 #include "net/base/net_util.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_interceptor.h"
 #include "storage/browser/blob/blob_storage_context.h"
 
 namespace content {
 
 namespace {
 
 int kUserDataKey;  // Key value is not important.
@@ skipping 15 matching lines @@
         request, network_delegate, resource_context_);
   }
 
  private:
   ResourceContext* resource_context_;
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerRequestInterceptor);
 };
 
 }  // namespace
 
 void ServiceWorkerRequestHandler::InitializeHandler(

[michaeln, 2015/07/30 01:35:08]

InitializeHandler could be an alternative place to adopt the
ServiceWorkerProviderHosts created in InitializeNavigationHandler. Provided
would could identify the original nav_request_id at this time, and lookup the
plznav_provider_host given that id.

I'm wondering if we can reduce the id value plumbing?

[Fabrice (no longer in Chrome), 2015/08/06 15:39:15]

We should avoid duplicating code. I think the proper way of handling this would
be to add if blocks with different code paths for PlzNavigate and current
navigation.

     net::URLRequest* request,
     ServiceWorkerContextWrapper* context_wrapper,
     storage::BlobStorageContext* blob_storage_context,
     int process_id,
     int provider_id,
     bool skip_service_worker,
     FetchRequestMode request_mode,
     FetchCredentialsMode credentials_mode,
     ResourceType resource_type,
     RequestContextType request_context_type,
@@ skipping 30 matching lines @@
                                           request_context_type,
                                           frame_type,
                                           blob_storage_context->AsWeakPtr(),
                                           body));
   if (!handler)
     return;
 
   request->SetUserData(&kUserDataKey, handler.release());
 }
 
+void ServiceWorkerRequestHandler::InitializeNavigationHandler(
+    net::URLRequest* request,
+    ServiceWorkerContextWrapper* context_wrapper,
+    storage::BlobStorageContext* blob_storage_context,
+    bool skip_service_worker,
+    bool is_main_frame,
+    scoped_refptr<ResourceRequestBody> body,
+    NavigationURLLoaderImplCore* loader) {
+  CHECK(base::CommandLine::ForCurrentProcess()->HasSwitch(
+      switches::kEnableBrowserSideNavigation));
+  if (!request->url().SchemeIsHTTPOrHTTPS())
+    return;
+  if (!context_wrapper || !context_wrapper->context())
+    return;
+  scoped_ptr<ServiceWorkerProviderHost> provider_host(
+      context_wrapper->context()->CreateNavigationProviderHost());
+  loader->SetServiceWorkerProviderHost(provider_host.Pass());
+  if (skip_service_worker) {
+    // TODO(horo): Does it work correctly?.
+    provider_host->SetDocumentUrl(net::SimplifyUrlForRequest(request->url()));
+    provider_host->SetTopmostFrameUrl(request->first_party_for_cookies());
+    provider_host->AddAllMatchingRegistrations();
+    return;
+  }
+  scoped_ptr<ServiceWorkerRequestHandler> handler(
+      loader->service_worker_provider_host()->CreateRequestHandler(
+          FETCH_REQUEST_MODE_NO_CORS, FETCH_CREDENTIALS_MODE_SAME_ORIGIN,
+          is_main_frame ? RESOURCE_TYPE_MAIN_FRAME : RESOURCE_TYPE_SUB_FRAME,
+          REQUEST_CONTEXT_TYPE_LOCATION,         // TODO(horo): pass in
+                                                 // FrameHostMsg_BeginNavigation
+          REQUEST_CONTEXT_FRAME_TYPE_TOP_LEVEL,  // TODO(horo): pass in
+                                                 // FrameHostMsg_BeginNavigation
+          blob_storage_context->AsWeakPtr(), body));
+  if (!handler)
+    return;
+  request->SetUserData(&kUserDataKey, handler.release());
+}
+
 ServiceWorkerRequestHandler* ServiceWorkerRequestHandler::GetHandler(
     net::URLRequest* request) {
   return static_cast<ServiceWorkerRequestHandler*>(
       request->GetUserData(&kUserDataKey));
 }
 
 scoped_ptr<net::URLRequestInterceptor>
 ServiceWorkerRequestHandler::CreateInterceptor(
     ResourceContext* resource_context) {
   return scoped_ptr<net::URLRequestInterceptor>(
@@ skipping 52 matching lines @@
     ResourceType resource_type)
     : context_(context),
       provider_host_(provider_host),
       blob_storage_context_(blob_storage_context),
       resource_type_(resource_type),
       old_process_id_(0),
       old_provider_id_(kInvalidServiceWorkerProviderId) {
 }
 
 }  // namespace content