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

Unified Diff: chrome/browser/supervised_user/child_accounts/child_account_service.cc

Issue 2334613003: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Changes from review by sky Created 4 years, 3 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: chrome/browser/supervised_user/child_accounts/child_account_service.cc
diff --git a/chrome/browser/supervised_user/child_accounts/child_account_service.cc b/chrome/browser/supervised_user/child_accounts/child_account_service.cc
index a624431c636a7aeb796c3dcd77843394bc5d8c91..45a363e5295c041c4ce0430a1cfc89fa88fc4849 100644
--- a/chrome/browser/supervised_user/child_accounts/child_account_service.cc
+++ b/chrome/browser/supervised_user/child_accounts/child_account_service.cc
@@ -149,18 +149,18 @@ bool ChildAccountService::SetActive(bool active) {
settings_service->SetLocalSetting(
supervised_users::kRecordHistoryIncludesSessionSync,
- base::WrapUnique(new base::FundamentalValue(false)));
+ base::MakeUnique<base::FundamentalValue>(false));
// In contrast to legacy SUs, child account SUs must sign in.
settings_service->SetLocalSetting(
supervised_users::kSigninAllowed,
- base::WrapUnique(new base::FundamentalValue(true)));
+ base::MakeUnique<base::FundamentalValue>(true));
// SafeSearch is controlled at the account level, so don't override it
// client-side.
settings_service->SetLocalSetting(
supervised_users::kForceSafeSearch,
- base::WrapUnique(new base::FundamentalValue(false)));
+ base::MakeUnique<base::FundamentalValue>(false));
#if !defined(OS_CHROMEOS)
// This is also used by user policies (UserPolicySigninService), but since

Powered by Google App Engine
This is Rietveld 408576698