Remove raw base::DictionaryValue::Set

device/geolocation/network_location_request.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 "device/geolocation/network_location_request.h"
 
 #include <stdint.h>
 
 #include <limits>
 #include <set>
 #include <string>
+#include <utility>
 
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
+#include "base/memory/ptr_util.h"
 #include "base/metrics/histogram.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/values.h"
 #include "device/geolocation/geoposition.h"
 #include "device/geolocation/location_arbitrator.h"
 #include "google_apis/google_api_keys.h"
 #include "net/base/escape.h"
 #include "net/base/load_flags.h"
@@ skipping 242 matching lines @@
 
   if (wifi_data.access_point_data.empty())
     return;
 
   typedef std::multiset<const AccessPointData*, AccessPointLess> AccessPointSet;
   AccessPointSet access_points_by_signal_strength;
 
   for (const auto& ap_data : wifi_data.access_point_data)
     access_points_by_signal_strength.insert(&ap_data);
 
-  base::ListValue* wifi_access_point_list = new base::ListValue();
+  auto wifi_access_point_list = base::MakeUnique<base::ListValue>();
   for (auto* ap_data : access_points_by_signal_strength) {
-    std::unique_ptr<base::DictionaryValue> wifi_dict(
-        new base::DictionaryValue());
+    auto wifi_dict = base::MakeUnique<base::DictionaryValue>();
     AddString("macAddress", base::UTF16ToUTF8(ap_data->mac_address),
               wifi_dict.get());
     AddInteger("signalStrength", ap_data->radio_signal_strength,
                wifi_dict.get());
     AddInteger("age", age_milliseconds, wifi_dict.get());
     AddInteger("channel", ap_data->channel, wifi_dict.get());
     AddInteger("signalToNoiseRatio", ap_data->signal_to_noise, wifi_dict.get());
     wifi_access_point_list->Append(std::move(wifi_dict));
   }
-  request->Set("wifiAccessPoints", wifi_access_point_list);
+  request->Set("wifiAccessPoints", std::move(wifi_access_point_list));
 }
 
 void FormatPositionError(const GURL& server_url,
                          const std::string& message,
                          Geoposition* position) {
   position->error_code = Geoposition::ERROR_CODE_POSITION_UNAVAILABLE;
   position->error_message = "Network location provider at '";
   position->error_message += server_url.GetOrigin().spec();
   position->error_message += "' : ";
   position->error_message += message;
@@ skipping 141 matching lines @@
 
   // Other fields are optional.
   GetAsDouble(*response_object, kAccuracyString, &position->accuracy);
 
   return true;
 }
 
 }  // namespace
 
 }  // namespace device