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

chrome/browser/background/background_application_list_model_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.
 
 // TODO(rickcam): Bug 73183: Add unit tests for image loading
 
 #include "chrome/browser/background/background_application_list_model.h"
 
 #include <stddef.h>
 
 #include <cstdlib>
 #include <memory>
 #include <set>
+#include <utility>
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
+#include "base/memory/ptr_util.h"
 #include "base/message_loop/message_loop.h"
 #include "base/stl_util.h"
 #include "build/build_config.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_service_test_base.h"
 #include "chrome/browser/extensions/permissions_updater.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/notification_types.h"
 #include "extensions/browser/extension_prefs.h"
@@ skipping 46 matching lines @@
 
 // Returns a barebones test Extension object with the specified |name|.  The
 // returned extension will include background permission if
 // |background_permission| is true.
 static scoped_refptr<Extension> CreateExtension(
     const std::string& name,
     bool background_permission) {
   base::DictionaryValue manifest;
   manifest.SetString(extensions::manifest_keys::kVersion, "1.0.0.0");
   manifest.SetString(extensions::manifest_keys::kName, name);
-  base::ListValue* permissions = new base::ListValue();
-  manifest.Set(extensions::manifest_keys::kPermissions, permissions);
+  auto permissions = base::MakeUnique<base::ListValue>();
   if (background_permission) {
     permissions->AppendString("background");
   }
+  manifest.Set(extensions::manifest_keys::kPermissions, std::move(permissions));
 
   std::string error;
   scoped_refptr<Extension> extension;
 
   extension = Extension::Create(bogus_file_pathname(name),
                                 extensions::Manifest::INTERNAL, manifest,
                                 Extension::NO_FLAGS, &error);
 
   // Cannot ASSERT_* here because that attempts an illegitimate return.
   // Cannot EXPECT_NE here because that assumes non-pointers unlike EXPECT_EQ
@@ skipping 294 matching lines @@
       case 2:
         TogglePermission(service(), &extensions, model.get(), &expected,
                          &count);
         break;
       default:
         NOTREACHED();
         break;
     }
   }
 }