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

chrome/browser/extensions/extension_protocols_unittest.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 <stddef.h>
 
 #include <memory>
 #include <string>

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

#include <utility>

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

Done.

 
 #include "base/command_line.h"
 #include "base/files/file_util.h"
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/run_loop.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/test/test_file_util.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/chrome_content_verifier_delegate.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/crx_file/id_util.h"
@@ skipping 72 matching lines @@
                         Extension::NO_FLAGS, &error));
   EXPECT_TRUE(extension.get()) << error;
   return extension;
 }
 
 scoped_refptr<Extension> CreateTestResponseHeaderExtension() {
   base::DictionaryValue manifest;
   manifest.SetString("name", "An extension with web-accessible resources");
   manifest.SetString("version", "2");
 
-  base::ListValue* web_accessible_list = new base::ListValue();
+  auto web_accessible_list = base::MakeUnique<base::ListValue>();
   web_accessible_list->AppendString("test.dat");
-  manifest.Set("web_accessible_resources", web_accessible_list);
+  manifest.Set("web_accessible_resources", std::move(web_accessible_list));
 
   base::FilePath path = GetTestPath("response_headers");
 
   std::string error;
   scoped_refptr<Extension> extension(
       Extension::Create(path, Manifest::UNPACKED, manifest,
                         Extension::NO_FLAGS, &error));
   EXPECT_TRUE(extension.get()) << error;
   return extension;
 }
@@ skipping 475 matching lines @@
   content_verifier_->OnExtensionLoaded(testing_profile_.get(), extension.get());
   // Wait for PostTask to ContentVerifierIOData::AddData() to finish.
   content::RunAllPendingInMessageLoop();
 
   // Request foo.js.
   EXPECT_EQ(net::OK, DoRequest(*extension, kFooJs));
   test_job_delegate.WaitForDoneReading(extension->id());
 }
 
 }  // namespace extensions