Stop passing raw pointers to base::Value API in c/b/ui

chrome/browser/ui/webui/identity_internals_ui.cc

 // Copyright 2013 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 "chrome/browser/ui/webui/identity_internals_ui.h"
 
 #include <memory>
 #include <set>
 #include <string>
 
@@ skipping 48 matching lines @@
   void RegisterMessages() override;
 
  private:
   // Gets the name of an extension referred to by |token_cache_key| as a string.
   const std::string GetExtensionName(
       const extensions::ExtensionTokenKey& token_cache_key);
 
   // Gets a list of scopes specified in |token_cache_key| and returns a pointer
   // to a ListValue containing the scopes. The caller gets ownership of the
   // returned object.
-  base::ListValue* GetScopes(
+  std::unique_ptr<base::ListValue> GetScopes(
       const extensions::ExtensionTokenKey& token_cache_key);
 
   // Gets a localized status of the access token in |token_cache_value|.
   const base::string16 GetStatus(
       const extensions::IdentityTokenCacheValue& token_cache_value);
 
   // Gets a string representation of an expiration time of the access token in
   // |token_cache_value|.
   const std::string GetExpirationTime(
       const extensions::IdentityTokenCacheValue& token_cache_value);
@@ skipping 94 matching lines @@
     const extensions::ExtensionTokenKey& token_cache_key) {
   const extensions::ExtensionRegistry* registry =
       extensions::ExtensionRegistry::Get(Profile::FromWebUI(web_ui()));
   const extensions::Extension* extension =
       registry->enabled_extensions().GetByID(token_cache_key.extension_id);
   if (!extension)
     return std::string();
   return extension->name();
 }
 
-base::ListValue* IdentityInternalsUIMessageHandler::GetScopes(
+std::unique_ptr<base::ListValue> IdentityInternalsUIMessageHandler::GetScopes(
     const extensions::ExtensionTokenKey& token_cache_key) {
-  base::ListValue* scopes_value = new base::ListValue();
+  auto scopes_value = base::MakeUnique<base::ListValue>();
   for (std::set<std::string>::const_iterator
            iter = token_cache_key.scopes.begin();
        iter != token_cache_key.scopes.end(); ++iter) {
     scopes_value->AppendString(*iter);
   }
   return scopes_value;
 }
 
 const base::string16 IdentityInternalsUIMessageHandler::GetStatus(
     const extensions::IdentityTokenCacheValue& token_cache_value) {
@@ skipping 125 matching lines @@
       IDR_IDENTITY_INTERNALS_JS);
   html_source->SetDefaultResource(IDR_IDENTITY_INTERNALS_HTML);
 
   content::WebUIDataSource::Add(Profile::FromWebUI(web_ui), html_source);
 
   web_ui->AddMessageHandler(
       base::MakeUnique<IdentityInternalsUIMessageHandler>());
 }
 
 IdentityInternalsUI::~IdentityInternalsUI() {}