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

components/signin/core/browser/about_signin_internals.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 "components/signin/core/browser/about_signin_internals.h"
 
 #include <stddef.h>
 
+#include <utility>
+
 #include "base/command_line.h"
 #include "base/hash.h"
 #include "base/i18n/time_formatting.h"
 #include "base/logging.h"
 #include "base/profiler/scoped_tracker.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/trace_event/trace_event.h"
 #include "build/build_config.h"
 #include "components/pref_registry/pref_registry_syncable.h"
@@ skipping 16 matching lines @@
   return base::UTF16ToUTF8(base::TimeFormatShortDateAndTime(time));
 }
 
 base::ListValue* AddSection(base::ListValue* parent_list,
                             const std::string& title) {
   std::unique_ptr<base::DictionaryValue> section(new base::DictionaryValue());
   base::ListValue* section_contents = new base::ListValue();
 
   section->SetString("title", title);
   section->Set("data", section_contents);
-  parent_list->Append(section.release());
+  parent_list->Append(std::move(section));
   return section_contents;
 }
 
 void AddSectionEntry(base::ListValue* section_list,
                      const std::string& field_name,
                      const std::string& field_status,
                      const std::string& field_time = "") {
   std::unique_ptr<base::DictionaryValue> entry(new base::DictionaryValue());
   entry->SetString("label", field_name);
   entry->SetString("status", field_status);
   entry->SetString("time", field_time);
-  section_list->Append(entry.release());
+  section_list->Append(std::move(entry));
 }
 
 void AddCookieEntry(base::ListValue* accounts_list,
                      const std::string& field_email,
                      const std::string& field_gaia_id,
                      const std::string& field_valid) {
   std::unique_ptr<base::DictionaryValue> entry(new base::DictionaryValue());
   entry->SetString("email", field_email);
   entry->SetString("gaia_id", field_gaia_id);
   entry->SetString("valid", field_valid);
-  accounts_list->Append(entry.release());
+  accounts_list->Append(std::move(entry));
 }
 
 std::string SigninStatusFieldToLabel(UntimedSigninStatusField field) {
   switch (field) {
     case ACCOUNT_ID:
       return "Account Id";
     case GAIA_ID:
       return "Gaia Id";
     case USERNAME:
       return "Username";
@@ skipping 589 matching lines @@
   }
 
   for(const std::string& account_id : accounts_in_token_service) {
     base::DictionaryValue* entry = new base::DictionaryValue();
     entry->SetString("accountId", account_id);
     account_info->Append(entry);
   }
 
   return signin_status;
 }