Reland: Stop passing raw pointers to base::Value API in c/b/chromeos and c/b/extensions

chrome/browser/chromeos/login/users/supervised_user_manager_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 "chrome/browser/chromeos/login/users/supervised_user_manager_impl.h"
 
 #include "base/files/file_path.h"
 #include "base/files/file_util.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/string_util.h"
@@ skipping 193 matching lines @@
   DictionaryPrefUpdate manager_update(local_state, kSupervisedUserManagers);
   DictionaryPrefUpdate manager_name_update(local_state,
                                            kSupervisedUserManagerNames);
   DictionaryPrefUpdate manager_email_update(
       local_state,
       kSupervisedUserManagerDisplayEmails);
 
   prefs_new_users_update->Insert(0,
                                  base::MakeUnique<base::Value>(local_user_id));
 
-  sync_id_update->SetWithoutPathExpansion(local_user_id,
-                                          new base::Value(sync_user_id));
+  sync_id_update->SetWithoutPathExpansion(
+      local_user_id, base::MakeUnique<base::Value>(sync_user_id));
   manager_update->SetWithoutPathExpansion(
-      local_user_id, new base::Value(manager->GetAccountId().GetUserEmail()));
+      local_user_id,
+      base::MakeUnique<base::Value>(manager->GetAccountId().GetUserEmail()));
   manager_name_update->SetWithoutPathExpansion(
-      local_user_id, new base::Value(manager->GetDisplayName()));
+      local_user_id, base::MakeUnique<base::Value>(manager->GetDisplayName()));
   manager_email_update->SetWithoutPathExpansion(
-      local_user_id, new base::Value(manager->display_email()));
+      local_user_id, base::MakeUnique<base::Value>(manager->display_email()));
 
   owner_->SaveUserDisplayName(AccountId::FromUserEmail(local_user_id),
                               display_name);
 
   g_browser_process->local_state()->CommitPendingWrite();
   return new_user;
 }
 
 std::string SupervisedUserManagerImpl::GetUserSyncId(const std::string& user_id)
     const {
@@ skipping 260 matching lines @@
 
   DictionaryPrefUpdate manager_name_update(local_state,
                                            kSupervisedUserManagerNames);
   for (base::DictionaryValue::Iterator it(*manager_ids); !it.IsAtEnd();
       it.Advance()) {
     std::string user_id;
     bool has_manager_id = it.value().GetAsString(&user_id);
     DCHECK(has_manager_id);
     if (user_id == manager_id) {
       manager_name_update->SetWithoutPathExpansion(
-          it.key(), new base::Value(new_display_name));
+          it.key(), base::MakeUnique<base::Value>(new_display_name));
     }
   }
 }
 
 SupervisedUserAuthentication* SupervisedUserManagerImpl::GetAuthentication() {
   return authentication_.get();
 }
 
 void SupervisedUserManagerImpl::LoadSupervisedUserToken(
     Profile* profile,
@@ skipping 10 matching lines @@
 }
 
 void SupervisedUserManagerImpl::ConfigureSyncWithToken(
     Profile* profile,
     const std::string& token) {
   if (!token.empty())
     SupervisedUserServiceFactory::GetForProfile(profile)->InitSync(token);
 }
 
 }  // namespace chromeos