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

chrome/browser/extensions/webstore_inline_installer_unittest.cc

 // Copyright (c) 2012 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 <vector>

[jdoerrie, 2017/04/06 14:25:51]

#include <utility>

[vabr (Chromium), 2017/04/07 20:40:41]

Done.

 
+#include "base/memory/ptr_util.h"
 #include "chrome/browser/extensions/webstore_inline_installer.h"
 #include "chrome/common/extensions/webstore_install_result.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "content/public/browser/web_contents.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 namespace extensions {
 
 namespace {
@@ skipping 71 matching lines @@
   scoped_refptr<TestWebstoreInlineInstaller> installer =
     new TestWebstoreInlineInstaller(web_contents_.get(), requestor_url);
   return installer->TestCheckRequestorPermitted(webstore_data);
 }
 
 // Simulates a test against a list of verified site strings from a Webstore
 // item's "verified_sites" manifest entry.
 bool WebstoreInlineInstallerTest::TestMultipleVerifiedSites(
     const std::string& requestor_url,
     const std::vector<std::string>& verified_sites) {
-  base::ListValue* sites = new base::ListValue();
+  auto sites = base::MakeUnique<base::ListValue>();
   for (std::vector<std::string>::const_iterator it = verified_sites.begin();
        it != verified_sites.end(); ++it) {
     sites->AppendString(*it);
   }
   base::DictionaryValue webstore_data;
-  webstore_data.Set("verified_sites", sites);
+  webstore_data.Set("verified_sites", std::move(sites));
 
   scoped_refptr<TestWebstoreInlineInstaller> installer =
     new TestWebstoreInlineInstaller(web_contents_.get(), requestor_url);
   return installer->TestCheckRequestorPermitted(webstore_data);
 }
 
 }  // namespace
 
 TEST_F(WebstoreInlineInstallerTest, DomainVerification) {
   // Exact domain match.
@@ skipping 104 matching lines @@
                                          verified_sites));
 
   EXPECT_FALSE(TestMultipleVerifiedSites("http://baz.example.com",
                                          verified_sites));
 
   EXPECT_FALSE(TestMultipleVerifiedSites("http://bar.example.com:456",
                                          verified_sites));
 }
 
 }  // namespace extensions