Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(218)

Unified Diff: components/drive/chromeos/change_list_loader.cc

Issue 2257793002: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/drive/chromeos/change_list_loader.cc
diff --git a/components/drive/chromeos/change_list_loader.cc b/components/drive/chromeos/change_list_loader.cc
index be3c96d2beb03edf26744fa3fbe97c431cdf1178..e2bb5b313c51ea1c06ee45db3c2fb5de9b30d57e 100644
--- a/components/drive/chromeos/change_list_loader.cc
+++ b/components/drive/chromeos/change_list_loader.cc
@@ -181,8 +181,8 @@ std::unique_ptr<base::ScopedClosureRunner> LoaderController::GetLock() {
DCHECK(thread_checker_.CalledOnValidThread());
++lock_count_;
- return base::WrapUnique(new base::ScopedClosureRunner(
- base::Bind(&LoaderController::Unlock, weak_ptr_factory_.GetWeakPtr())));
+ return base::MakeUnique<base::ScopedClosureRunner>(
+ base::Bind(&LoaderController::Unlock, weak_ptr_factory_.GetWeakPtr()));
}
void LoaderController::ScheduleRun(const base::Closure& task) {
@@ -283,8 +283,7 @@ void AboutResourceLoader::UpdateAboutResourceAfterGetAbout(
for (size_t i = 0; i < callbacks.size(); ++i) {
callbacks[i].Run(
- status,
- base::WrapUnique(new google_apis::AboutResource(*about_resource)));
+ status, base::MakeUnique<google_apis::AboutResource>(*about_resource));
}
}
@@ -512,8 +511,8 @@ void ChangeListLoader::LoadChangeListFromServer(int64_t start_changestamp) {
change_feed_fetcher_->Run(
base::Bind(&ChangeListLoader::LoadChangeListFromServerAfterLoadChangeList,
weak_ptr_factory_.GetWeakPtr(),
- base::Passed(base::WrapUnique(new google_apis::AboutResource(
- *about_resource_loader_->cached_about_resource()))),
+ base::Passed(base::MakeUnique<google_apis::AboutResource>(
+ *about_resource_loader_->cached_about_resource())),
is_delta_update));
}
« no previous file with comments | « components/domain_reliability/monitor.cc ('k') | components/drive/chromeos/file_system/download_operation.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698