[Sync] Rewriting ".reset(new" pattern to use "= base::MakeUnique" instead.

components/sync/model_impl/attachments/attachment_service_impl_unittest.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/sync/model_impl/attachments/attachment_service_impl.h"
 
 #include <algorithm>
 #include <map>
 #include <utility>
 #include <vector>
@@ skipping 131 matching lines @@
     download_requests.insert(std::make_pair(id, callback));
   }
 
   // Multiple requests to download will be active at the same time.
   // RespondToDownload should respond to only one of them.
   void RespondToDownload(const AttachmentId& id, const DownloadResult& result) {
     ASSERT_TRUE(download_requests.find(id) != download_requests.end());
     std::unique_ptr<Attachment> attachment;
     if (result == DOWNLOAD_SUCCESS) {
       scoped_refptr<base::RefCountedString> data = new base::RefCountedString();
-      attachment.reset(new Attachment(Attachment::CreateFromParts(id, data)));
+      attachment =
+          base::MakeUnique<Attachment>(Attachment::CreateFromParts(id, data));
     }
     base::ThreadTaskRunnerHandle::Get()->PostTask(
         FROM_HERE,
         base::Bind(download_requests[id], result, base::Passed(&attachment)));
 
     download_requests.erase(id);
   }
 
   std::map<AttachmentId, DownloadCallback> download_requests;
 
@@ skipping 68 matching lines @@
     std::unique_ptr<AttachmentStore> attachment_store =
         AttachmentStore::CreateMockStoreForTest(
             std::move(attachment_store_backend));
 
     if (uploader.get()) {
       attachment_uploader_ = uploader->AsWeakPtr();
     }
     if (downloader.get()) {
       attachment_downloader_ = downloader->AsWeakPtr();
     }
-    attachment_service_.reset(new AttachmentServiceImpl(
+    attachment_service_ = base::MakeUnique<AttachmentServiceImpl>(
         attachment_store->CreateAttachmentStoreForSync(), std::move(uploader),
         std::move(downloader), delegate, base::TimeDelta::FromMinutes(1),
-        base::TimeDelta::FromMinutes(8)));
+        base::TimeDelta::FromMinutes(8));
 
     std::unique_ptr<base::MockTimer> timer_to_pass(
         new base::MockTimer(false, false));
     mock_timer_ = timer_to_pass.get();
     attachment_service_->SetTimerForTest(std::move(timer_to_pass));
   }
 
   AttachmentService* attachment_service() { return attachment_service_.get(); }
 
   base::MockTimer* mock_timer() { return mock_timer_; }
@@ skipping 372 matching lines @@
   net::NetworkChangeNotifier::NotifyObserversOfNetworkChangeForTests(
       net::NetworkChangeNotifier::CONNECTION_WIFI);
   RunLoop();
 
   // No longer in backoff.
   ASSERT_TRUE(mock_timer()->IsRunning());
   ASSERT_EQ(base::TimeDelta(), mock_timer()->GetCurrentDelay());
 }
 
 }  // namespace syncer