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

chrome/test/chromedriver/logging.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/test/chromedriver/logging.h"
 
 #include <stddef.h>
 #include <stdint.h>
 #include <stdio.h>
 
+#include <utility>
+
 #include "base/command_line.h"
 #include "base/json/json_reader.h"
 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/strings/stringprintf.h"
 #include "base/time/time.h"
 #include "build/build_config.h"
 #include "chrome/test/chromedriver/capabilities.h"
 #include "chrome/test/chromedriver/chrome/console_logger.h"
 #include "chrome/test/chromedriver/chrome/status.h"
@@ skipping 160 matching lines @@
     return;
 
   std::unique_ptr<base::DictionaryValue> log_entry_dict(
       new base::DictionaryValue());
   log_entry_dict->SetDouble("timestamp",
                             static_cast<int64_t>(timestamp.ToJsTime()));
   log_entry_dict->SetString("level", LevelToName(level));
   if (!source.empty())
     log_entry_dict->SetString("source", source);
   log_entry_dict->SetString("message", message);
-  entries_->Append(log_entry_dict.release());
+  entries_->Append(std::move(log_entry_dict));
 }
 
 const std::string& WebDriverLog::type() const {
   return type_;
 }
 
 void WebDriverLog::set_min_level(Level min_level) {
   min_level_ = min_level;
 }
 
@@ skipping 87 matching lines @@
   logs.push_back(browser_log);
   // If the level is OFF, don't even bother listening for DevTools events.
   if (browser_log_level != Log::kOff)
     devtools_listeners.push_back(new ConsoleLogger(browser_log));
 
   out_logs->swap(logs);
   out_devtools_listeners->swap(devtools_listeners);
   out_command_listeners->swap(command_listeners);
   return Status(kOk);
 }