PlzNavigate: Make ServiceWorker work with PlzNavigate.

content/browser/loader/navigation_url_loader_impl_core.h

 // 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.
 
 #ifndef CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_IMPL_CORE_H_
 #define CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_IMPL_CORE_H_
 
 #include "base/basictypes.h"
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "content/browser/loader/navigation_url_loader_impl.h"
 
 namespace net {
 class URLRequest;
 struct RedirectInfo;
 }
 
 namespace content {
 
 class FrameTreeNode;
 class NavigationResourceHandler;
 class ResourceContext;
 class ResourceHandler;
 class ResourceRequestBody;
+class ServiceWorkerContext;
+class ServiceWorkerContextWrapper;
+class ServiceWorkerProviderHost;
 class StreamHandle;
 struct ResourceResponse;
 
 // The IO-thread counterpart to the NavigationURLLoaderImpl. It lives on the IO
 // thread and is owned by the UI-thread NavigationURLLoaderImpl.
 // NavigationURLLoaderImplCore interacts with the ResourceDispatcherHost stack
 // and forwards signals back to the loader on the UI thread.
 class NavigationURLLoaderImplCore {
  public:
   // Creates a new NavigationURLLoaderImplCore that forwards signals back to
   // |loader| on the UI thread.
   explicit NavigationURLLoaderImplCore(
-      const base::WeakPtr<NavigationURLLoaderImpl>& loader);
+      const base::WeakPtr<NavigationURLLoaderImpl>& loader,
+      ServiceWorkerContext* service_worker_context);
   ~NavigationURLLoaderImplCore();
 
   // Starts the request.
   void Start(ResourceContext* resource_context,
              int frame_tree_node_id,
              scoped_ptr<NavigationRequestInfo> request_info);
 
   // Follows the current pending redirect.
   void FollowRedirect();
 
   void set_resource_handler(NavigationResourceHandler* resource_handler) {
     resource_handler_ = resource_handler;
   }
 
+  ServiceWorkerProviderHost* service_worker_provider_host() {
+    return service_worker_provider_host_.get();
+  }
+  void SetServiceWorkerProviderHost(scoped_ptr<ServiceWorkerProviderHost>);

[carlosk, 2015/08/27 14:26:42]

nit: new line here.

[Fabrice (no longer in Chrome), 2015/08/28 15:40:15]

Spacing between getters and setters in headers is so inconsistent across the
code base, I am not sure what is the expected way to do it.

+
   // Notifies |loader_| on the UI thread that the request was redirected.
   void NotifyRequestRedirected(const net::RedirectInfo& redirect_info,
                                ResourceResponse* response);
 
   // Notifies |loader_| on the UI thread that the response started.
   void NotifyResponseStarted(ResourceResponse* response,
                              scoped_ptr<StreamHandle> body);
 
   // Notifies |loader_| on the UI thread that the request failed.
   void NotifyRequestFailed(bool in_cache, int net_error);
 
  private:
   base::WeakPtr<NavigationURLLoaderImpl> loader_;
+  scoped_refptr<ServiceWorkerContextWrapper> service_worker_context_;
   NavigationResourceHandler* resource_handler_;
+  scoped_ptr<ServiceWorkerProviderHost> service_worker_provider_host_;
 
   DISALLOW_COPY_AND_ASSIGN(NavigationURLLoaderImplCore);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_LOADER_NAVIGATION_URL_LOADER_IMPL_CORE_H_