Stop passing raw pointers to DictionaryValue::Set, part 2

chrome/browser/chromeos/file_manager/file_browser_handlers.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 "chrome/browser/chromeos/file_manager/file_browser_handlers.h"
 
 #include <stddef.h>
 #include <algorithm>
 #include <set>
 #include <utility>
@@ skipping 374 matching lines @@
   }
 
   SetupHandlerHostFileAccessPermissions(
       file_definition_list.get(), extension_.get(), handler_pid);
 
   std::unique_ptr<base::ListValue> event_args(new base::ListValue());
   event_args->AppendString(action_id_);
   auto details = base::MakeUnique<base::DictionaryValue>();
   // Get file definitions. These will be replaced with Entry instances by
   // dispatchEvent() method from event_binding.js.
-  base::ListValue* file_entries = new base::ListValue();
-  details->Set("entries", file_entries);
-  event_args->Append(std::move(details));
+  auto file_entries = base::MakeUnique<base::ListValue>();
 
   for (EntryDefinitionList::const_iterator iter =
            entry_definition_list->begin();
        iter != entry_definition_list->end();
        ++iter) {
     auto file_def = base::MakeUnique<base::DictionaryValue>();
     file_def->SetString("fileSystemName", iter->file_system_name);
     file_def->SetString("fileSystemRoot", iter->file_system_root_url);
     file_def->SetString("fileFullPath",
                         "/" + iter->full_path.AsUTF8Unsafe());
     file_def->SetBoolean("fileIsDirectory", iter->is_directory);
     file_entries->Append(std::move(file_def));
   }
 
+  details->Set("entries", std::move(file_entries));
+  event_args->Append(std::move(details));
   std::unique_ptr<extensions::Event> event(new extensions::Event(
       extensions::events::FILE_BROWSER_HANDLER_ON_EXECUTE,
       "fileBrowserHandler.onExecute", std::move(event_args)));
   event->restrict_to_browser_context = profile_;
   router->DispatchEventToExtension(extension_->id(), std::move(event));
 
   ExecuteDoneOnUIThread(true);
 }
 
 void FileBrowserHandlerExecutor::SetupHandlerHostFileAccessPermissions(
@@ skipping 107 matching lines @@
       if (common_handlers.empty())
         return FileBrowserHandlerList();
     }
   }
 
   return common_handlers;
 }
 
 }  // namespace file_browser_handlers
 }  // namespace file_manager