Patch on top of https://codereview.chromium.org/433793002/

Source/modules/serviceworkers/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 "config.h"
 #include "Request.h"
 
 #include "bindings/core/v8/Dictionary.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/fetch/FetchUtils.h"
@@ skipping 169 matching lines @@
         exceptionState.throwTypeError("Invalid URL");
         return 0;
     }
     // "3. Set |request|'s url to |parsedURL|."
     request->setURL(parsedURL);
     // "4. Set |fallbackMode| to CORS."
     // "5. Set |fallbackCredentials| to omit."
     return createRequestWithRequestData(request, RequestInit(context, init, exceptionState), FetchRequestData::CORSMode, FetchRequestData::OmitCredentials, exceptionState);
 }
 
-Request* Request::create(ExecutionContext* context, Request* input, ExceptionState& exceptionState)
+Request* Request::create(ExecutionContext* context, const Request* input, ExceptionState& exceptionState)
 {
     return create(context, input, Dictionary(), exceptionState);
 }
 
-Request* Request::create(ExecutionContext* context, Request* input, const Dictionary& init, ExceptionState& exceptionState)
+Request* Request::create(ExecutionContext* context, const Request* input, const Dictionary& init, ExceptionState& exceptionState)
 {
     // "1. Let |request| be |input|'s associated request, if |input| is a
     // Request object, and a new request otherwise."
     // "2. Set |request| to a restricted copy of itself."
     FetchRequestData* request(input->request()->createRestrictedCopy(context, SecurityOrigin::create(context->url())));
     // "3. Let |fallbackMode| be null."
     // "4. Let |fallbackCredentials| be null."
     // Instead of using null as a special fallback value, just pass the current
     // mode and credentials; it has the same effect.
     const FetchRequestData::Mode currentMode = request->mode();
@@ skipping 88 matching lines @@
         return "omit";
     case FetchRequestData::SameOriginCredentials:
         return "same-origin";
     case FetchRequestData::IncludeCredentials:
         return "include";
     }
     ASSERT_NOT_REACHED();
     return "";
 }
 
-void Request::populateWebServiceWorkerRequest(WebServiceWorkerRequest& webRequest)
+void Request::populateWebServiceWorkerRequest(WebServiceWorkerRequest& webRequest) const
 {
     webRequest.setMethod(method());
     webRequest.setURL(m_request->url());
     m_headers->forEach(adoptPtrWillBeNoop(new FillWebRequestHeaders(&webRequest)));
     webRequest.setReferrer(m_request->referrer().referrer().referrer, static_cast<WebReferrerPolicy>(m_request->referrer().referrer().referrerPolicy));
     // FIXME: How can we set isReload properly? What is the correct place to load it in to the Request object? We should investigate the right way
     // to plumb this information in to here.
 }
 
 void Request::setBodyBlobHandle(PassRefPtr<BlobDataHandle>blobHandle)
 {
     m_request->setBlobDataHandle(blobHandle);
 }
 
 void Request::trace(Visitor* visitor)
 {
     visitor->trace(m_request);
     visitor->trace(m_headers);
     visitor->trace(m_fetchBodyStream);
 }
 
 } // namespace blink