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

content/browser/service_worker/service_worker_controllee_request_handler.cc

 // 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.
 
 #include "content/browser/service_worker/service_worker_controllee_request_handler.h"
 
 #include "content/browser/service_worker/service_worker_context_core.h"
 #include "content/browser/service_worker/service_worker_metrics.h"
 #include "content/browser/service_worker/service_worker_provider_host.h"
 #include "content/browser/service_worker/service_worker_registration.h"
@@ skipping 28 matching lines @@
     if (is_main_resource_load_)
       provider_host_->active_version()->ScheduleUpdate();
     else
       provider_host_->active_version()->DeferScheduledUpdate();
   }
 }
 
 net::URLRequestJob* ServiceWorkerControlleeRequestHandler::MaybeCreateJob(
     net::URLRequest* request,
     net::NetworkDelegate* network_delegate) {
-  if (!context_ || !provider_host_ ||
-      request->load_flags() & net::LOAD_BYPASS_CACHE) {
+  if (!context_ || !provider_host_) {
     // We can't do anything other than to fall back to network.
     job_ = NULL;
     return NULL;
   }
 
+  if (request->load_flags() & net::LOAD_BYPASS_CACHE) {
+    if (is_main_resource_load_ && provider_host_) {

[michaeln, 2014/07/11 19:48:53]

dont think we can get here if !provider_host_ so the test for it can be removed

+      provider_host_->SetDocumentUrl(
+          net::SimplifyUrlForRequest(request->url()));
+    }
+    job_ = NULL;
+    return NULL;
+  }
+
   // This may get called multiple times for original and redirect requests:
   // A. original request case: job_ is null, no previous location info.
   // B. redirect or restarted request case:
   //  a) job_ is non-null if the previous location was forwarded to SW.
   //  b) job_ is null if the previous location was fallback.
   //  c) job_ is non-null if additional restart was required to fall back.
 
   // We've come here by restart, we already have original request and it
   // tells we should fallback to network. (Case B-c)
   if (job_.get() && job_->ShouldFallbackToNetwork()) {
@@ skipping 62 matching lines @@
 }
 
 void ServiceWorkerControlleeRequestHandler::PrepareForSubResource() {
   DCHECK(job_.get());
   DCHECK(context_);
   DCHECK(provider_host_->active_version());
   job_->ForwardToServiceWorker();
 }
 
 }  // namespace content