Prerender: Confirm ServiceWorkers are invoked for NoState Prefetch

content/browser/service_worker/service_worker_context_request_handler.cc

Index: content/browser/service_worker/service_worker_context_request_handler.cc
diff --git a/content/browser/service_worker/service_worker_context_request_handler.cc b/content/browser/service_worker/service_worker_context_request_handler.cc
index 36d2875f96a41aa976bbdedc181211b96bcdcead..869e58e790c86fdc85db2d8d12d5a8e5cab94596 100644
--- a/content/browser/service_worker/service_worker_context_request_handler.cc
+++ b/content/browser/service_worker/service_worker_context_request_handler.cc
@@ -55,6 +55,12 @@ net::URLRequestJob* ServiceWorkerContextRequestHandler::MaybeCreateJob(
     return NULL;
   }
 
+  // TODO(mattcary): still not clear if this is correct or not.
+  // Skip creation if job is prefetch.
+  if (request->load_flags() & net::LOAD_PREFETCH) {
+    return NULL;

[falken, 2016/12/27 01:27:00]

Returning null here just means that SW machinery will not intercept the request,
and it will go through the usual resource loading/network stack.

Also, ServiceWorkerContextRequestHandler is created for requests done from the
SW which means either:
- the SW calling fetch()
- the request for the SW script itself to start the SW

Since requests initiated by the SW calling fetch() are *not* intercepted by any
SW, they go straight to the normal network stack, hence the return null in line
55.

So ServiceWorkerContextRequestHandler only tries to intercept requests for the
SW script to start the worker. In this case, if we return null here, the SW
script load won't go through the expected path (which would reading or writing
to the SW script storage), and will instead go through the network stack. The
renderer will get the response for the script and start the worker anyway, which
is an unexpected case. In fact I expect this would cause a CHECK later when the
browser discovers a service worker is running without going through the expected
load path (I think the other null returns past line 55 also would cause this
CHECK, it's a long standing bug I'm trying to track down:
https://bugs.chromium.org/p/chromium/issues/detail?id=485900.

If the goal is to drop the job, I think we'd need to return a job that responds
to the request with an error, or cancel the request.

Also there is another request handler called
ServiceWorkerControlleeRequestHandler, which is for intercepting requests from a
"client" (typically a controlled page) that could be intercepted by the SW. I
suspect if you want requests involving NoPrefetch to not be intercepted by a SW,
you'll need code there too.

+  }
+
   if (ShouldAddToScriptCache(request->url())) {
     ServiceWorkerRegistration* registration =
         context_->GetLiveRegistration(version_->registration_id());