Remove raw base::DictionaryValue::Set in //content

net/log/net_log.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 "net/log/net_log.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/logging.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 
 namespace net {
 
 namespace {
 
 std::unique_ptr<base::Value> NetLogBoolCallback(
     const char* name,
@@ skipping 170 matching lines @@
     return #label;
 #include "net/log/net_log_event_type_list.h"
 #undef EVENT_TYPE
     default:
       NOTREACHED();
       return NULL;
   }
 }
 
 // static
-base::Value* NetLog::GetEventTypesAsValue() {
-  std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
+std::unique_ptr<base::Value> NetLog::GetEventTypesAsValue() {
+  auto dict = base::MakeUnique<base::DictionaryValue>();
   for (int i = 0; i < static_cast<int>(NetLogEventType::COUNT); ++i) {
     dict->SetInteger(EventTypeToString(static_cast<NetLogEventType>(i)), i);
   }
-  return dict.release();
+  return std::move(dict);
 }
 
 // static
 const char* NetLog::SourceTypeToString(NetLogSourceType source) {
   switch (source) {
 #define SOURCE_TYPE(label)      \
   case NetLogSourceType::label: \
     return #label;
 #include "net/log/net_log_source_type_list.h"
 #undef SOURCE_TYPE
     default:
       NOTREACHED();
       return NULL;
   }
 }
 
 // static
-base::Value* NetLog::GetSourceTypesAsValue() {
-  std::unique_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
+std::unique_ptr<base::Value> NetLog::GetSourceTypesAsValue() {
+  auto dict = base::MakeUnique<base::DictionaryValue>();
   for (int i = 0; i < static_cast<int>(NetLogSourceType::COUNT); ++i) {
     dict->SetInteger(SourceTypeToString(static_cast<NetLogSourceType>(i)), i);
   }
-  return dict.release();
+  return std::move(dict);
 }
 
 // static
 const char* NetLog::EventPhaseToString(NetLogEventPhase phase) {
   switch (phase) {
     case NetLogEventPhase::BEGIN:
       return "PHASE_BEGIN";
     case NetLogEventPhase::END:
       return "PHASE_END";
     case NetLogEventPhase::NONE:
@@ skipping 49 matching lines @@
   NetLogEntryData entry_data(type, source, phase, base::TimeTicks::Now(),
                              parameters_callback);
 
   // Notify all of the log observers.
   base::AutoLock lock(lock_);
   for (auto* observer : observers_)
     observer->OnAddEntryData(entry_data);
 }
 
 }  // namespace net