[ServiceWorker] Introduce skip_service_worker flag and remove LOAD_BYPASS_CACHE check.

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 64442b5bf60a80ec5135b3ec0dd4aa0aed6b2e44..a7474207c08333932ea96f1e2bf9889b4176c09e 100644
--- a/content/browser/service_worker/service_worker_controllee_request_handler.cc
+++ b/content/browser/service_worker/service_worker_controllee_request_handler.cc
@@ -55,15 +55,6 @@ net::URLRequestJob* ServiceWorkerControlleeRequestHandler::MaybeCreateJob(
     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: