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

media/cast/logging/stats_event_subscriber.cc

 // Copyright 2014 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 "media/cast/logging/stats_event_subscriber.h"
 
 #include <algorithm>
 #include <cmath>
+#include <utility>
 
 #include "base/format_macros.h"
 #include "base/logging.h"
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
 
 #define STAT_ENUM_TO_STRING(enum) \
   case enum:                      \
     return #enum
 
@@ skipping 45 matching lines @@
 
 std::unique_ptr<base::ListValue>
 StatsEventSubscriber::SimpleHistogram::GetHistogram() const {
   std::unique_ptr<base::ListValue> histo(new base::ListValue);
 
   std::unique_ptr<base::DictionaryValue> bucket(new base::DictionaryValue);
 
   if (buckets_.front()) {
     bucket->SetInteger(base::StringPrintf("<%" PRId64, min_),
                        buckets_.front());
-    histo->Append(bucket.release());
+    histo->Append(std::move(bucket));
   }
 
   for (size_t i = 1; i < buckets_.size() - 1; i++) {
     if (!buckets_[i])
       continue;
     bucket.reset(new base::DictionaryValue);
     int64_t lower = min_ + (i - 1) * width_;
     int64_t upper = lower + width_ - 1;
     bucket->SetInteger(
         base::StringPrintf("%" PRId64 "-%" PRId64, lower, upper),
         buckets_[i]);
-    histo->Append(bucket.release());
+    histo->Append(std::move(bucket));
   }
 
   if (buckets_.back()) {
     bucket.reset(new base::DictionaryValue);
     bucket->SetInteger(base::StringPrintf(">=%" PRId64, max_),
                        buckets_.back());
-    histo->Append(bucket.release());
+    histo->Append(std::move(bucket));
   }
   return histo;
 }
 
 StatsEventSubscriber::StatsEventSubscriber(
     EventMediaType event_media_type,
     base::TickClock* clock,
     ReceiverTimeOffsetEstimator* offset_estimator)
     : event_media_type_(event_media_type),
       clock_(clock),
@@ skipping 643 matching lines @@
     : event_counter(0), sum_size(0) {}
 StatsEventSubscriber::PacketLogStats::~PacketLogStats() {}
 
 StatsEventSubscriber::FrameInfo::FrameInfo() : encoded(false) {
 }
 StatsEventSubscriber::FrameInfo::~FrameInfo() {
 }
 
 }  // namespace cast
 }  // namespace media