Fetch API: Don't exclude the URL fragment in Request#url

third_party/WebKit/Source/modules/fetch/Request.cpp

 // 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 "modules/fetch/Request.h"
 
 #include "bindings/core/v8/Dictionary.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/loader/ThreadableLoader.h"
@@ skipping 475 matching lines @@
     : Request(scriptState, request, Headers::create(request->headerList())) {
   m_headers->setGuard(Headers::RequestGuard);
 }
 
 String Request::method() const {
   // "The method attribute's getter must return request's method."
   return m_request->method();
 }
 
 KURL Request::url() const {
-  // The url attribute's getter must return request's url, serialized with the
-  // exclude fragment flag set.
-  if (!m_request->url().hasFragmentIdentifier())
-    return m_request->url();
-  KURL url(m_request->url());
-  url.removeFragmentIdentifier();
-  return url;
+  return m_request->url();
 }
 
 String Request::context() const {
   // "The context attribute's getter must return request's context"
   switch (m_request->context()) {
     case WebURLRequest::RequestContextUnspecified:
       return "";
     case WebURLRequest::RequestContextAudio:
       return "audio";
     case WebURLRequest::RequestContextBeacon:
@@ skipping 176 matching lines @@
 }
 
 bool Request::hasBody() const {
   return bodyBuffer();
 }
 
 void Request::populateWebServiceWorkerRequest(
     WebServiceWorkerRequest& webRequest) const {
   webRequest.setMethod(method());
   webRequest.setRequestContext(m_request->context());
-  // This strips off the fragment part.
-  webRequest.setURL(url());
+
+  // Strip off the fragment part of URL. So far, all users of
+  // WebServiceWorkerRequest expect the fragment to be excluded.
+  KURL url(m_request->url());
+  if (m_request->url().hasFragmentIdentifier())
+    url.removeFragmentIdentifier();
+  webRequest.setURL(url);
 
   const FetchHeaderList* headerList = m_headers->headerList();
   for (size_t i = 0, size = headerList->size(); i < size; ++i) {
     const FetchHeaderList::Header& header = headerList->entry(i);
     webRequest.appendHeader(header.first, header.second);
   }
 
   webRequest.setReferrer(
       m_request->referrerString(),
       static_cast<WebReferrerPolicy>(m_request->getReferrerPolicy()));
@@ skipping 22 matching lines @@
       V8HiddenValue::internalBodyBuffer(scriptState->isolate()), bodyBuffer);
 }
 
 DEFINE_TRACE(Request) {
   Body::trace(visitor);
   visitor->trace(m_request);
   visitor->trace(m_headers);
 }
 
 }  // namespace blink