Move TestingPrefService to use unique_ptr<Value>

components/certificate_transparency/ct_policy_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 "components/certificate_transparency/ct_policy_manager.h"
 
 #include <iterator>
 
 #include "base/run_loop.h"
 #include "base/sequenced_task_runner.h"
 #include "base/single_thread_task_runner.h"
 #include "base/test/test_message_loop.h"
 #include "base/values.h"
 #include "components/certificate_transparency/pref_names.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/testing_pref_service.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace certificate_transparency {
 
 namespace {
 
-base::ListValue* ListValueFromStrings(const std::vector<const char*>& strings) {
+std::unique_ptr<base::ListValue> ListValueFromStrings(
+    const std::vector<const char*>& strings) {
   std::unique_ptr<base::ListValue> result(new base::ListValue);
   for (auto* const str : strings) {
     result->AppendString(str);
   }
-  return result.release();
+  return result;
 }
 
 class CTPolicyManagerTest : public ::testing::Test {
  public:
   CTPolicyManagerTest() : message_loop_(base::MessageLoop::TYPE_IO) {}
 
  protected:
   base::TestMessageLoop message_loop_;
   TestingPrefServiceSimple pref_service_;
 };
@@ skipping 227 matching lines @@
             delegate->IsCTRequiredForHost("sub.example.com"));
 
   // And it should still be possible to get the delegate, even after calling
   // Shutdown().
   EXPECT_TRUE(manager.GetDelegate());
 }
 
 }  // namespace
 
 }  // namespace certificate_transparency