Remove raw base::DictionaryValue::Set

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/memory/ptr_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
 
 #define STAT_ENUM_TO_STRING(enum) \
   case enum:                      \
     return #enum
 
 namespace media {
 namespace cast {
 
@@ skipping 197 matching lines @@
   if (last_event_time_.is_null()) {
     last_event_time_ = timestamp;
   } else {
     last_event_time_ = std::max(last_event_time_, timestamp);
   }
 }
 
 std::unique_ptr<base::DictionaryValue> StatsEventSubscriber::GetStats() const {
   StatsMap stats_map;
   GetStatsInternal(&stats_map);
-  std::unique_ptr<base::DictionaryValue> ret(new base::DictionaryValue);
+  auto ret = base::MakeUnique<base::DictionaryValue>();
 
-  std::unique_ptr<base::DictionaryValue> stats(new base::DictionaryValue);
+  auto stats = base::MakeUnique<base::DictionaryValue>();
   for (StatsMap::const_iterator it = stats_map.begin(); it != stats_map.end();
        ++it) {
     // Round to 3 digits after the decimal point.
     stats->SetDouble(CastStatToString(it->first),
                      round(it->second * 1000.0) / 1000.0);
   }
 
   // Populate all histograms.
   for (HistogramMap::const_iterator it = histograms_.begin();
        it != histograms_.end();
        ++it) {
-    stats->Set(CastStatToString(it->first),
-               it->second->GetHistogram().release());
+    stats->Set(CastStatToString(it->first), it->second->GetHistogram());
   }
 
   ret->Set(event_media_type_ == AUDIO_EVENT ? "audio" : "video",
-           stats.release());
+           std::move(stats));
 
   return ret;
 }
 
 void StatsEventSubscriber::Reset() {
   DCHECK(thread_checker_.CalledOnValidThread());
 
   frame_stats_.clear();
   packet_stats_.clear();
   total_capture_latency_ = base::TimeDelta();
@@ skipping 488 matching lines @@
     : event_counter(0), sum_size(0) {}
 StatsEventSubscriber::PacketLogStats::~PacketLogStats() {}
 
 StatsEventSubscriber::FrameInfo::FrameInfo() : encoded(false) {
 }
 StatsEventSubscriber::FrameInfo::~FrameInfo() {
 }
 
 }  // namespace cast
 }  // namespace media