base::ListValue::Append cleanup: pass unique_ptr instead of the released pointer.

chrome/browser/extensions/menu_manager.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/extensions/menu_manager.h"
 
 #include <algorithm>
 #include <tuple>
 #include <utility>
 
@@ skipping 74 matching lines @@
     if (!item)
       continue;
     items.push_back(item);
   }
   return items;
 }
 
 std::unique_ptr<base::Value> MenuItemsToValue(const MenuItem::List& items) {
   std::unique_ptr<base::ListValue> list(new base::ListValue());
   for (size_t i = 0; i < items.size(); ++i)
-    list->Append(items[i]->ToValue().release());
+    list->Append(items[i]->ToValue());
   return std::unique_ptr<base::Value>(list.release());
 }
 
 bool GetStringList(const base::DictionaryValue& dict,
                    const std::string& key,
                    std::vector<std::string>* out) {
   if (!dict.HasKey(key))
     return true;
 
   const base::ListValue* list = NULL;
@@ skipping 572 matching lines @@
 
   // Add the tab info to the argument list.
   // No tab info in a platform app.
   if (!extension || !extension->is_platform_app()) {
     // Note: web_contents are NULL in unit tests :(
     if (web_contents) {
       int frame_id = ExtensionApiFrameIdMap::GetFrameId(render_frame_host);
       if (frame_id != ExtensionApiFrameIdMap::kInvalidFrameId)
         properties->SetInteger("frameId", frame_id);
 
-      args->Append(
-          ExtensionTabUtil::CreateTabObject(web_contents)->ToValue().release());
+      args->Append(ExtensionTabUtil::CreateTabObject(web_contents)->ToValue());
     } else {
       args->Append(new base::DictionaryValue());
     }
   }
 
   if (item->type() == MenuItem::CHECKBOX ||
       item->type() == MenuItem::RADIO) {
     bool was_checked = item->checked();
     properties->SetBoolean("wasChecked", was_checked);
 
@@ skipping 286 matching lines @@
 bool MenuItem::Id::operator!=(const Id& other) const {
   return !(*this == other);
 }
 
 bool MenuItem::Id::operator<(const Id& other) const {
   return std::tie(incognito, extension_key, uid, string_uid) <
     std::tie(other.incognito, other.extension_key, other.uid, other.string_uid);
 }
 
 }  // namespace extensions