Move TestingPrefService to use unique_ptr<Value>

chrome/browser/chromeos/arc/arc_session_manager_unittest.cc

 // Copyright 2016 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 <memory>
 #include <string>
 #include <tuple>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 482 matching lines @@
 TEST_P(ArcSessionManagerPolicyTest, SkippingTerms) {
   sync_preferences::TestingPrefServiceSyncable* const prefs =
       profile()->GetTestingPrefService();
 
   // Backup-restore and location-service prefs are off by default.
   EXPECT_FALSE(prefs->GetBoolean(prefs::kArcSignedIn));
   EXPECT_FALSE(prefs->GetBoolean(prefs::kArcTermsAccepted));
 
   // Enable ARC through user pref or by policy, according to the test parameter.
   if (arc_enabled_pref_managed())
-    prefs->SetManagedPref(prefs::kArcEnabled, new base::Value(true));
+    prefs->SetManagedPref(prefs::kArcEnabled,
+                          base::MakeUnique<base::Value>(true));
   else
     prefs->SetBoolean(prefs::kArcEnabled, true);
   EXPECT_TRUE(IsArcPlayStoreEnabledForProfile(profile()));
 
   // Assign test values to the prefs.
   if (backup_restore_pref_value().is_bool()) {
     prefs->SetManagedPref(prefs::kArcBackupRestoreEnabled,
-                          backup_restore_pref_value().DeepCopy());
+                          backup_restore_pref_value().CreateDeepCopy());
   }
   if (location_service_pref_value().is_bool()) {
     prefs->SetManagedPref(prefs::kArcLocationServiceEnabled,
-                          location_service_pref_value().DeepCopy());
+                          location_service_pref_value().CreateDeepCopy());
   }
 
   arc_session_manager()->SetProfile(profile());
   arc_session_manager()->RequestEnable();
 
   // Terms of Service are skipped iff ARC is enabled by policy and both
   // ArcBackupRestoreEnabled and ArcLocationServiceEnabled are managed.
   const bool expected_terms_skipping = arc_enabled_pref_managed() &&
                                        backup_restore_pref_value().is_bool() &&
                                        location_service_pref_value().is_bool();
@@ skipping 27 matching lines @@
   prefs->RemoveManagedPref(prefs::kArcEnabled);
   WaitForDataRemoved(ArcSessionManager::State::STOPPED);
   arc_session_manager()->Shutdown();
 }
 
 TEST_P(ArcSessionManagerPolicyTest, ReenableManagedArc) {
   sync_preferences::TestingPrefServiceSyncable* const prefs =
       profile()->GetTestingPrefService();
 
   // Set ARC to be managed.
-  prefs->SetManagedPref(prefs::kArcEnabled, new base::Value(true));
+  prefs->SetManagedPref(prefs::kArcEnabled,
+                        base::MakeUnique<base::Value>(true));
   EXPECT_TRUE(arc::IsArcPlayStoreEnabledForProfile(profile()));
 
   arc_session_manager()->SetProfile(profile());
   arc_session_manager()->RequestEnable();
   EXPECT_TRUE(arc_session_manager()->enable_requested());
 
   // Simulate close OptIn. Session manager should stop.
   SetArcPlayStoreEnabledForProfile(profile(), false);
   EXPECT_TRUE(arc::IsArcPlayStoreEnabledForProfile(profile()));
   EXPECT_FALSE(arc_session_manager()->enable_requested());
@@ skipping 111 matching lines @@
     GetFakeUserManager()->set_current_user_new(true);
 
     CreateLoginDisplayHost();
 
     if (IsManagedUser()) {
       policy::ProfilePolicyConnector* const connector =
           policy::ProfilePolicyConnectorFactory::GetForBrowserContext(
               profile());
       connector->OverrideIsManagedForTesting(true);
 
-      profile()->GetTestingPrefService()->SetManagedPref(prefs::kArcEnabled,
-                                                         new base::Value(true));
+      profile()->GetTestingPrefService()->SetManagedPref(
+          prefs::kArcEnabled, base::MakeUnique<base::Value>(true));
     }
 
     arc_session_manager()->SetProfile(profile());
     if (IsArcPlayStoreEnabledForProfile(profile()))
       arc_session_manager()->RequestEnable();
   }
 
   void TearDown() override {
     // Correctly stop service.
     arc_session_manager()->Shutdown();
@@ skipping 101 matching lines @@
     // For managed case we handle closing outside of
     // ArcPlayStoreEnabledPreferenceHandler. So it session turns to STOPPED.
     EXPECT_EQ(ArcSessionManager::State::STOPPED,
               arc_session_manager()->state());
     // Managed user's preference should not be overwritten.
     EXPECT_TRUE(IsArcPlayStoreEnabledForProfile(profile()));
   }
 }
 
 }  // namespace arc