Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(430)

Side by Side Diff: content/child/web_url_loader_impl.cc

Issue 2140523002: Remove WebURLRequest::initialize() and simplify WebURLRequest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: mutable ref Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/child/web_url_loader_impl.h" 5 #include "content/child/web_url_loader_impl.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <memory> 10 #include <memory>
(...skipping 589 matching lines...) Expand 10 before | Expand all | Expand 10 after
600 600
601 TRACE_EVENT_WITH_FLOW0( 601 TRACE_EVENT_WITH_FLOW0(
602 "loading", "WebURLLoaderImpl::Context::OnReceivedRedirect", 602 "loading", "WebURLLoaderImpl::Context::OnReceivedRedirect",
603 this, TRACE_EVENT_FLAG_FLOW_IN | TRACE_EVENT_FLAG_FLOW_OUT); 603 this, TRACE_EVENT_FLAG_FLOW_IN | TRACE_EVENT_FLAG_FLOW_OUT);
604 604
605 WebURLResponse response; 605 WebURLResponse response;
606 PopulateURLResponse(request_.url(), info, &response, 606 PopulateURLResponse(request_.url(), info, &response,
607 request_.reportRawHeaders()); 607 request_.reportRawHeaders());
608 608
609 WebURLRequest new_request; 609 WebURLRequest new_request;
610 new_request.initialize();
611 PopulateURLRequestForRedirect( 610 PopulateURLRequestForRedirect(
612 request_, redirect_info, referrer_policy_, 611 request_, redirect_info, referrer_policy_,
613 info.was_fetched_via_service_worker 612 info.was_fetched_via_service_worker
614 ? blink::WebURLRequest::SkipServiceWorker::None 613 ? blink::WebURLRequest::SkipServiceWorker::None
615 : blink::WebURLRequest::SkipServiceWorker::All, 614 : blink::WebURLRequest::SkipServiceWorker::All,
616 &new_request); 615 &new_request);
617 616
618 client_->willFollowRedirect(loader_, new_request, response); 617 client_->willFollowRedirect(loader_, new_request, response);
619 request_ = new_request; 618 request_ = new_request;
620 619
(...skipping 606 matching lines...) Expand 10 before | Expand all | Expand 10 after
1227 response->clearHTTPHeaderField(webStringName); 1226 response->clearHTTPHeaderField(webStringName);
1228 while (response_headers->EnumerateHeader(&iterator, name, &value)) { 1227 while (response_headers->EnumerateHeader(&iterator, name, &value)) {
1229 response->addHTTPHeaderField(webStringName, 1228 response->addHTTPHeaderField(webStringName,
1230 WebString::fromLatin1(value)); 1229 WebString::fromLatin1(value));
1231 } 1230 }
1232 } 1231 }
1233 return true; 1232 return true;
1234 } 1233 }
1235 1234
1236 } // namespace content 1235 } // namespace content
OLDNEW
« no previous file with comments | « components/plugins/renderer/mobile_youtube_plugin.cc ('k') | content/child/web_url_loader_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698