Adjust URLFetcher::Create API so that object is returned as scoped_ptr.

chrome/browser/chromeos/attestation/attestation_ca_client.cc

 // Copyright (c) 2013 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 "chrome/browser/chromeos/attestation/attestation_ca_client.h"
 
 #include <string>
 
 #include "chrome/browser/browser_process.h"
 #include "net/base/load_flags.h"
@@ skipping 59 matching lines @@
   DCHECK(result) << "Invalid fetcher setting.";
 
   // Run the callback last because it may delete |this|.
   callback.Run(true, response);
 }
 
 void AttestationCAClient::FetchURL(const std::string& url,
                                    const std::string& request,
                                    const DataCallback& on_response) {
   // The first argument allows the use of TestURLFetcherFactory in tests.
-  net::URLFetcher* fetcher = net::URLFetcher::Create(0,
-                                                     GURL(url),
-                                                     net::URLFetcher::POST,
-                                                     this);
+  net::URLFetcher* fetcher =
+      net::URLFetcher::Create(0, GURL(url), net::URLFetcher::POST, this)
+          .release();

[Ryan Sleevi, 2015/04/30 18:23:44]

Ideally, you'd move the .release() to line 88 and have this be a scoped_ptr<>

   fetcher->SetRequestContext(g_browser_process->system_request_context());
   fetcher->SetLoadFlags(net::LOAD_DO_NOT_SEND_COOKIES |
                         net::LOAD_DO_NOT_SAVE_COOKIES |
                         net::LOAD_DISABLE_CACHE);
   fetcher->SetUploadData(kMimeContentType, request);
   pending_requests_[fetcher] = on_response;
   fetcher->Start();
 }
 
 }  // namespace attestation
 }  // namespace chromeos