Reduce string copies in GURL creation

components/html_viewer/web_url_loader_impl.cc

 // 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 "components/html_viewer/web_url_loader_impl.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <utility>
 
@@ skipping 31 matching lines @@
   if (base::StartsWith(status_line.get(), "HTTP/1.1",
                        base::CompareCase::SENSITIVE))
     return blink::WebURLResponse::HTTPVersion_1_1;
 
   return blink::WebURLResponse::HTTPVersionUnknown;
 }
 
 blink::WebURLResponse ToWebURLResponse(const URLResponsePtr& url_response) {
   blink::WebURLResponse result;
   result.initialize();
-  result.setURL(GURL(url_response->url));
+  result.setURL(GURL(url_response->url.get()));
   result.setMIMEType(blink::WebString::fromUTF8(url_response->mime_type));
   result.setTextEncodingName(blink::WebString::fromUTF8(url_response->charset));
   result.setHTTPVersion(StatusLineToHTTPVersion(url_response->status_line));
   result.setHTTPStatusCode(url_response->status_code);
   result.setExpectedContentLength(-1);  // Not available.
 
   // TODO(darin): Initialize timing properly.
   blink::WebURLLoadTiming timing;
   timing.initialize();
   result.setLoadTiming(timing);
@@ skipping 36 matching lines @@
   url_request->auto_follow_redirects = true;
   URLResponsePtr url_response;
   url_loader_->Start(std::move(url_request),
                      [&url_response](URLResponsePtr url_response_result) {
                        url_response = std::move(url_response_result);
                      });
   url_loader_.WaitForIncomingResponse();
   if (url_response->error) {
     error.domain = WebString::fromUTF8(net::kErrorDomain);
     error.reason = url_response->error->code;
-    error.unreachableURL = GURL(url_response->url);
+    error.unreachableURL = GURL(url_response->url.get());
     return;
   }
 
   response = ToWebURLResponse(url_response);
   std::string body;
   mojo::common::BlockingCopyToString(std::move(url_response->body), &body);
   data.assign(body.data(), body.length());
 }
 
 void WebURLLoaderImpl::loadAsynchronously(const blink::WebURLRequest& request,
@@ skipping 50 matching lines @@
                  blink::WebURLRequest(),
                  base::Passed(&failed_response)));
 }
 
 void WebURLLoaderImpl::setDefersLoading(bool defers_loading) {
   NOTIMPLEMENTED();
 }
 
 void WebURLLoaderImpl::OnReceivedResponse(const blink::WebURLRequest& request,
                                           URLResponsePtr url_response) {
-  url_ = GURL(url_response->url);
+  url_ = GURL(url_response->url.get());
 
   if (url_response->error) {
     OnReceivedError(std::move(url_response));
   } else if (url_response->redirect_url) {
     OnReceivedRedirect(request, std::move(url_response));
   } else {
     base::WeakPtr<WebURLLoaderImpl> self(weak_factory_.GetWeakPtr());
     client_->didReceiveResponse(this, ToWebURLResponse(url_response));
 
     // We may have been deleted during didReceiveResponse.
     if (!self)
       return;
 
     // Start streaming data
     response_body_stream_ = std::move(url_response->body);
     ReadMore();
   }
 }
 
 void WebURLLoaderImpl::OnReceivedError(URLResponsePtr url_response) {
   blink::WebURLError web_error;
   web_error.domain = blink::WebString::fromUTF8(net::kErrorDomain);
   web_error.reason = url_response->error->code;
-  web_error.unreachableURL = GURL(url_response->url);
+  web_error.unreachableURL = GURL(url_response->url.get());
   web_error.staleCopyInCache = false;
   web_error.isCancellation =
       url_response->error->code == net::ERR_ABORTED ? true : false;
 
   client_->didFail(this, web_error);
 }
 
 void WebURLLoaderImpl::OnReceivedRedirect(const blink::WebURLRequest& request,
                                           URLResponsePtr url_response) {
   // TODO(erg): setFirstPartyForCookies() and setHTTPReferrer() are unset here.
   blink::WebURLRequest new_request;
   new_request.initialize();
-  new_request.setURL(GURL(url_response->redirect_url));
+  new_request.setURL(GURL(url_response->redirect_url.get()));
   new_request.setDownloadToFile(request.downloadToFile());
   new_request.setRequestContext(request.requestContext());
   new_request.setFrameType(request.frameType());
   new_request.setSkipServiceWorker(request.skipServiceWorker());
   new_request.setFetchRequestMode(request.fetchRequestMode());
   new_request.setFetchCredentialsMode(request.fetchCredentialsMode());
   new_request.setHTTPReferrer(
       WebString::fromUTF8(url_response->redirect_referrer),
       referrer_policy_);
 
@@ skipping 87 matching lines @@
 void WebURLLoaderImpl::OnResponseBodyStreamReady(MojoResult result) {
   ReadMore();
 }
 
 void WebURLLoaderImpl::setLoadingTaskRunner(
     blink::WebTaskRunner* web_task_runner) {
   // TODO(alexclarke): Consider hooking this up.
 }
 
 }  // namespace html_viewer