Implement ServiceWorkerFetchDispatcher::MaybeStartNavigationPreload().

content/browser/service_worker/service_worker_url_request_job.h

Index: content/browser/service_worker/service_worker_url_request_job.h
diff --git a/content/browser/service_worker/service_worker_url_request_job.h b/content/browser/service_worker/service_worker_url_request_job.h
index 278f250a2ad32685f7a7027e0bb94423486229a3..cecc7c3ee4ec4b61b8515ca28df7a9484cc1d672 100644
--- a/content/browser/service_worker/service_worker_url_request_job.h
+++ b/content/browser/service_worker/service_worker_url_request_job.h
@@ -33,12 +33,12 @@
 
 namespace net {
 class IOBuffer;
-}
+}  // namespace net
 
 namespace storage {
 class BlobDataHandle;
 class BlobStorageContext;
-}
+}  // namespace storage
 
 namespace content {
 
@@ -95,6 +95,7 @@ class CONTENT_EXPORT ServiceWorkerURLRequestJob : public net::URLRequestJob {
       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;
@@ -284,6 +285,7 @@ class CONTENT_EXPORT ServiceWorkerURLRequestJob : public net::URLRequestJob {
   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);