[ServiceWorker] Introduce skip_service_worker flag and remove LOAD_BYPASS_CACHE check.

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 37 matching lines @@
 
 net::URLRequestJob* ServiceWorkerControlleeRequestHandler::MaybeCreateJob(
     net::URLRequest* request,
     net::NetworkDelegate* network_delegate) {
   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_->SetDocumentUrl(

[michaeln, 2014/09/03 20:18:58]

this change puts the browser side into an odd unexpected state where it doesn't
know the document url of a ProviderHost object. i'm not sure this is the best
approach, i think this will regress an old bug.

https://code.google.com/p/chromium/issues/detail?id=392405
https://codereview.chromium.org/362183002

-          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 108 matching lines @@
 }
 
 void ServiceWorkerControlleeRequestHandler::PrepareForSubResource() {
   DCHECK(job_.get());
   DCHECK(context_);
   DCHECK(provider_host_->active_version());
   job_->ForwardToServiceWorker();
 }
 
 }  // namespace content