ServiceWorker: Strip away fragments from document url

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 148ce443174481e11b288cbd84d5374aa2bbea08..241820045d076080ecbffe9831e0872e8e8e80d7 100644
--- a/content/browser/service_worker/service_worker_controllee_request_handler.cc
+++ b/content/browser/service_worker/service_worker_controllee_request_handler.cc
@@ -10,6 +10,7 @@
 #include "content/browser/service_worker/service_worker_url_request_job.h"
 #include "content/browser/service_worker/service_worker_utils.h"
 #include "content/common/service_worker/service_worker_types.h"
+#include "net/base/net_util.h"
 #include "net/url_request/url_request.h"
 
 namespace content {
@@ -52,10 +53,13 @@ net::URLRequestJob* ServiceWorkerControlleeRequestHandler::MaybeCreateJob(
   // It's for original request (A) or redirect case (B-a or B-b).
   DCHECK(!job_.get() || job_->ShouldForwardToServiceWorker());
 
+  // Strip away reference fragments.
+  const GURL& stripped_url(net::SimplifyUrlForRequest(request->url()));

[michaeln, 2014/05/30 23:38:47]

const GURL&, doesn't look right here

i'd put this line inside the PrepareForMainResource method to be closer to where
its used, that way it would be more clear that this value is not used for the
RequestJob

[nhiroki, 2014/06/02 06:26:08]

Ah, it's a slip of the finger. Good catch!
Done.

+
   job_ = new ServiceWorkerURLRequestJob(request, network_delegate,
                                         provider_host_);
   if (ServiceWorkerUtils::IsMainResourceType(resource_type_))
-    PrepareForMainResource(request->url());
+    PrepareForMainResource(stripped_url);
   else
     PrepareForSubResource();