Rewrite simple uses of base::ListValue::Append(base::Value*) on CrOS.

chrome/browser/ui/webui/chromeos/choose_mobile_network_ui.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/browser/ui/webui/chromeos/choose_mobile_network_ui.h"
 
 #include <stddef.h>
 
+#include <memory>
 #include <set>
 #include <string>
+#include <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_piece.h"
 #include "base/values.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/browser_resources.h"
 #include "chrome/grit/generated_resources.h"
 #include "chromeos/network/device_state.h"
 #include "chromeos/network/network_device_handler.h"
 #include "chromeos/network/network_event_log.h"
@@ skipping 155 matching lines @@
   const DeviceState::CellularScanResults& scan_results =
       cellular->scan_results();
   std::set<std::string> network_ids;
   for (DeviceState::CellularScanResults::const_iterator it =
       scan_results.begin(); it != scan_results.end(); ++it) {
     // We need to remove duplicates from the list because same network with
     // different technologies are listed multiple times. But ModemManager
     // Register API doesn't allow technology to be specified so just show unique
     // network in UI.
     if (network_ids.insert(it->network_id).second) {
-      base::DictionaryValue* network = new base::DictionaryValue();
+      auto network = base::MakeUnique<base::DictionaryValue>();
       network->SetString(kNetworkIdProperty, it->network_id);
       if (!it->long_name.empty())
         network->SetString(kOperatorNameProperty, it->long_name);
       else if (!it->short_name.empty())
         network->SetString(kOperatorNameProperty, it->short_name);
       else
         network->SetString(kOperatorNameProperty, it->network_id);
       network->SetString(kStatusProperty, it->status);
       network->SetString(kTechnologyProperty, it->technology);
-      networks_list_.Append(network);
+      networks_list_.Append(std::move(network));
     }
   }
   if (is_page_ready_) {
     web_ui()->CallJavascriptFunctionUnsafe(kJsApiShowNetworks, networks_list_);
     networks_list_.Clear();
     has_pending_results_ = false;
   } else {
     has_pending_results_ = true;
   }
 }
@@ skipping 54 matching lines @@
     : WebUIController(web_ui) {
   ChooseMobileNetworkHandler* handler = new ChooseMobileNetworkHandler();
   web_ui->AddMessageHandler(handler);
   // Set up the "chrome://choose-mobile-network" source.
   Profile* profile = Profile::FromWebUI(web_ui);
   content::WebUIDataSource::Add(
       profile, CreateChooseMobileNetworkUIHTMLSource());
 }
 
 }  // namespace chromeos