ServiceWorker: Set provider host document URL for no-cache load.

content/browser/service_worker/service_worker_controllee_request_handler.cc

Index: content/browser/service_worker/service_worker_controllee_request_handler.cc
diff --git a/content/browser/service_worker/service_worker_controllee_request_handler.cc b/content/browser/service_worker/service_worker_controllee_request_handler.cc
index 4ae7ef20a9a7fe3cac9364d8d68b24ea3da8b356..98a5208f374ab2d471dddbb4fc42c9c80875a34c 100644
--- a/content/browser/service_worker/service_worker_controllee_request_handler.cc
+++ b/content/browser/service_worker/service_worker_controllee_request_handler.cc
@@ -45,6 +45,9 @@ ServiceWorkerControlleeRequestHandler::
 net::URLRequestJob* ServiceWorkerControlleeRequestHandler::MaybeCreateJob(
     net::URLRequest* request,
     net::NetworkDelegate* network_delegate) {
+  if (is_main_resource_load_ && provider_host_)
+    provider_host_->SetDocumentUrl(request->url());
+
   if (!context_ || !provider_host_ ||
       request->load_flags() & net::LOAD_BYPASS_CACHE) {

[michaeln, 2014/07/02 18:40:30]

might make sense to split the BYPASS condition out of this first early exit case
since its really of a different nature and we want to do other stuff in that
case too

if (BYPASS) {
  if (main_load)
    SetDocUrl(SimplifyUrlForRequest(url));
  job_= NULL;
  reuturn NULL;
}

     // We can't do anything other than to fall back to network.
@@ -97,7 +100,6 @@ void ServiceWorkerControlleeRequestHandler::PrepareForMainResource(
   provider_host_->SetWaitingVersion(NULL);
 
   GURL stripped_url = net::SimplifyUrlForRequest(url);
-  provider_host_->SetDocumentUrl(stripped_url);

[michaeln, 2014/07/02 18:40:30]

probably should continue to pass in a stripped_url

   context_->storage()->FindRegistrationForDocument(
       stripped_url,
       base::Bind(&self::DidLookupRegistrationForMainResource,