Remove unnecessary calls to GURL()

net/url_request/url_request.cc

Index: net/url_request/url_request.cc
diff --git a/net/url_request/url_request.cc b/net/url_request/url_request.cc
index 368b91ae3eff103afdb77ae5bb48c5e20bc9eeb9..be5816f405002435ce5f180aa9354ec85a67dec5 100644
--- a/net/url_request/url_request.cc
+++ b/net/url_request/url_request.cc
@@ -642,8 +642,8 @@ void URLRequest::StartJob(URLRequestJob* job) {
 
   response_info_.was_cached = false;
 
-  if (GURL(referrer_) != URLRequestJob::ComputeReferrerForRedirect(
-                             referrer_policy_, referrer_, url())) {
+  if (URLRequestJob::ComputeReferrerForRedirect(referrer_policy_, referrer_,
+                                                url()) != referrer_) {

[esprehn, 2016/11/09 21:09:25]

ditto, why do we not need to canonicalize referrer_?

[cfredric, 2016/11/09 22:12:46]

I had thought it was already canonicalized here. Changed back.

Is it worth saving the canonicalized value so we can reuse it in the next if
condition? Or is that memory we don't want to use?

     if (!network_delegate_ ||
         !network_delegate_->CancelURLRequestWithPolicyViolatingReferrerHeader(
             *this, url(), GURL(referrer_))) {