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

sync/internal_api/attachments/attachment_uploader_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 "sync/internal_api/public/attachments/attachment_uploader_impl.h"
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 192 matching lines @@
     const std::string& access_token,
     const base::Time& expiration_time) {
   DCHECK(CalledOnValidThread());
   if (is_stopped_) {
     return;
   }
 
   DCHECK_EQ(access_token_request_.get(), request);
   access_token_request_.reset();
   access_token_ = access_token;
-  fetcher_.reset(
-      net::URLFetcher::Create(upload_url_, net::URLFetcher::POST, this));
+  fetcher_ = net::URLFetcher::Create(upload_url_, net::URLFetcher::POST, this);
   ConfigureURLFetcherCommon(fetcher_.get(), access_token_, raw_store_birthday_,
                             model_type_, url_request_context_getter_.get());
 
   const uint32_t crc32c = attachment_.GetCrc32c();
   fetcher_->AddExtraRequestHeader(base::StringPrintf(
       "X-Goog-Hash: crc32c=%s", FormatCrc32cHash(crc32c).c_str()));
 
   // TODO(maniscalco): Is there a better way?  Copying the attachment data into
   // a string feels wrong given how large attachments may be (several MBs).  If
   // we may end up switching from URLFetcher to URLRequest, this copy won't be
@@ skipping 170 matching lines @@
 
 void AttachmentUploaderImpl::Base64URLSafeEncode(const std::string& input,
                                                  std::string* output) {
   base::Base64Encode(input, output);
   base::ReplaceChars(*output, "+", "-", output);
   base::ReplaceChars(*output, "/", "_", output);
   base::TrimString(*output, "=", output);
 }
 
 }  // namespace syncer