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

Unified Diff: net/url_request/url_request.cc

Issue 1151843002: DO NOT LAND Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More. Created 5 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: net/url_request/url_request.cc
diff --git a/net/url_request/url_request.cc b/net/url_request/url_request.cc
index 7496bb3914489c59be088f643db92a61a1b2562a..b7009744bc946cae9fc597269be8192dfbfb58d1 100644
--- a/net/url_request/url_request.cc
+++ b/net/url_request/url_request.cc
@@ -621,8 +621,7 @@ void URLRequest::StartJob(URLRequestJob* job) {
response_info_.was_cached = false;
- if (GURL(referrer_) != URLRequestJob::ComputeReferrerForRedirect(
- referrer_policy_, referrer_, url())) {
+ if (referrer_ != URLRequestJob::ComputeReferrerForRedirect(referrer_policy_, referrer_, url())) {
if (!network_delegate_ ||
!network_delegate_->CancelURLRequestWithPolicyViolatingReferrerHeader(
*this, url(), GURL(referrer_))) {
@@ -950,10 +949,10 @@ int URLRequest::Redirect(const RedirectInfo& redirect_info) {
//
// TODO(jww): This is probably layering violation and should be refactored
// into //content. See https://crbug.com/471397.
- if (redirect_info.new_url.GetOrigin() != url().GetOrigin() &&
+ if (url::Origin(redirect_info.new_url) != url::Origin(url()) &&
extra_request_headers_.HasHeader(HttpRequestHeaders::kOrigin)) {
extra_request_headers_.SetHeader(HttpRequestHeaders::kOrigin,
- url::Origin().string());
+ url::Origin().serialize());
}
referrer_ = redirect_info.new_referrer;

Powered by Google App Engine
This is Rietveld 408576698