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

Unified Diff: components/sync/driver/fake_sync_client.cc

Issue 2256733004: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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/sync/driver/fake_sync_client.cc
diff --git a/components/sync/driver/fake_sync_client.cc b/components/sync/driver/fake_sync_client.cc
index 9e5ee432f0289e124e7f3125d6ad30b5905d4fda..1b3702a2a95b3de8bd41c9a94ba9d2ed96d33c36 100644
--- a/components/sync/driver/fake_sync_client.cc
+++ b/components/sync/driver/fake_sync_client.cc
@@ -23,7 +23,7 @@ void DummyRegisterPlatformTypesCallback(SyncService* sync_service,
FakeSyncClient::FakeSyncClient()
: model_type_service_(nullptr),
factory_(nullptr),
- sync_service_(base::WrapUnique(new FakeSyncService())) {
+ sync_service_(base::MakeUnique<FakeSyncService>()) {
// Register sync preferences and set them to "Sync everything" state.
sync_driver::SyncPrefs::RegisterProfilePrefs(pref_service_.registry());
sync_driver::SyncPrefs sync_prefs(GetPrefService());
@@ -32,8 +32,7 @@ FakeSyncClient::FakeSyncClient()
}
FakeSyncClient::FakeSyncClient(SyncApiComponentFactory* factory)
- : factory_(factory),
- sync_service_(base::WrapUnique(new FakeSyncService())) {
+ : factory_(factory), sync_service_(base::MakeUnique<FakeSyncService>()) {
sync_driver::SyncPrefs::RegisterProfilePrefs(pref_service_.registry());
}

Powered by Google App Engine
This is Rietveld 408576698