Implement ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload().

content/browser/service_worker/service_worker_url_request_job.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_SERVICE_WORKER_SERVICE_WORKER_URL_REQUEST_JOB_H_
 #define CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_URL_REQUEST_JOB_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 15 matching lines @@
 #include "net/http/http_byte_range.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_job.h"
 #include "net/url_request/url_request_status.h"
 #include "storage/common/blob_storage/blob_storage_constants.h"
 #include "third_party/WebKit/public/platform/modules/serviceworker/WebServiceWorkerResponseType.h"
 #include "url/gurl.h"
 
 namespace net {
 class IOBuffer;
-}
+}  // namespace net
 
 namespace storage {
 class BlobDataHandle;
 class BlobStorageContext;
-}
+}  // namespace storage
 
 namespace content {
 
 class ResourceContext;
 class ResourceRequestBodyImpl;
 class ServiceWorkerBlobReader;
 class ServiceWorkerStreamReader;
 class ServiceWorkerContextCore;
 class ServiceWorkerFetchDispatcher;
 class ServiceWorkerProviderHost;
@@ skipping 36 matching lines @@
       base::WeakPtr<storage::BlobStorageContext> blob_storage_context,
       const ResourceContext* resource_context,
       FetchRequestMode request_mode,
       FetchCredentialsMode credentials_mode,
       FetchRedirectMode redirect_mode,
       ResourceType resource_type,
       RequestContextType request_context_type,
       RequestContextFrameType frame_type,
       scoped_refptr<ResourceRequestBodyImpl> body,
       ServiceWorkerFetchType fetch_type,
+      const MojoURLLoaderFactoryGetter& url_loader_factory_getter,

[dcheng, 2016/10/20 02:35:50]

For me, it's a little confusing that this is a null callback most of the time.
Maybe some sort of comment on the member that indicates when this is used would
be helpful for maintainability in the future.

[horo, 2016/10/20 03:05:14]

Done.

       Delegate* delegate);
 
   ~ServiceWorkerURLRequestJob() override;
 
   const ResourceContext* resource_context() const { return resource_context_; }
 
   // Sets the response type.
   // When an in-flight request possibly needs CORS check, use
   // FallbackToNetworkOrRenderer. This method will decide whether the request
   // can directly go to the network or should fallback to a renderer to send
@@ skipping 169 matching lines @@
   bool response_is_in_cache_storage_ = false;
   std::string response_cache_storage_cache_name_;
 
   ServiceWorkerHeaderList cors_exposed_header_names_;
 
   std::unique_ptr<BlobConstructionWaiter> blob_construction_waiter_;
 
   bool worker_already_activated_ = false;
   EmbeddedWorkerStatus initial_worker_status_ = EmbeddedWorkerStatus::STOPPED;
 
+  const MojoURLLoaderFactoryGetter url_loader_factory_getter_;
   base::WeakPtrFactory<ServiceWorkerURLRequestJob> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerURLRequestJob);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_SERVICE_WORKER_SERVICE_WORKER_URL_REQUEST_JOB_H_