Remove redundant copy of blink::ResourceRequest for resource load

Bug: 632580
Change-Id: I9c72ae2f0df48124d417682c97a1bd2c35b1f6a0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1864499
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Auto-Submit: Makoto Shimazu <shimazu@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#712957}

third_party/blink/renderer/platform/loader/fetch/resource_fetcher.cc

@@ -1891,7 +1891,6 @@ bool ResourceFetcher::StartLoad(Resource* resource) {
   DCHECK(resource);
   DCHECK(resource->StillNeedsLoad());
 
-  ResourceRequest request(resource->GetResourceRequest());
   ResourceLoader* loader = nullptr;
 
   {
@@ -1906,6 +1905,7 @@ bool ResourceFetcher::StartLoad(Resource* resource) {
       return false;
     }
 
+    const auto& request = resource->GetResourceRequest();
     ResourceResponse response;
 
     resource->VirtualTimePauser().PauseVirtualTime();
@@ -1915,10 +1915,6 @@ bool ResourceFetcher::StartLoad(Resource* resource) {
           resource->InspectorId(), request, response, resource->GetType(),
           resource->Options().initiator_info);
     }
-    // TODO(shaochuan): Saving modified ResourceRequest back to |resource|,
-    // remove once dispatchWillSendRequest() takes const ResourceRequest.
-    // crbug.com/632580
-    resource->SetResourceRequest(request);
 
     using QuotaType = decltype(inflight_keepalive_bytes_);
     QuotaType size = 0;