Reland: Stop passing raw pointers to base::Value API in c/b/chromeos and c/b/extensions

chrome/browser/devtools/devtools_targets_ui.cc

Index: chrome/browser/devtools/devtools_targets_ui.cc
diff --git a/chrome/browser/devtools/devtools_targets_ui.cc b/chrome/browser/devtools/devtools_targets_ui.cc
index 300848ea7f576675700aabba08003c5687bf674b..f45c2fbf353c7d13b5ac2caf149a229ea76eff5d 100644
--- a/chrome/browser/devtools/devtools_targets_ui.cc
+++ b/chrome/browser/devtools/devtools_targets_ui.cc
@@ -251,12 +251,13 @@ void LocalTargetsUIHandler::SendTargets(
       list_value.Append(base::WrapUnique(descriptor));
     } else {
       base::DictionaryValue* parent = id_to_descriptor[parent_id];
-      base::ListValue* guests = NULL;
-      if (!parent->GetList(kGuestList, &guests)) {
-        guests = new base::ListValue();
-        parent->Set(kGuestList, guests);
+      base::ListValue* guests_weak = NULL;
+      if (!parent->GetList(kGuestList, &guests_weak)) {
+        auto guests = base::MakeUnique<base::ListValue>();
+        guests_weak = guests.get();
+        parent->Set(kGuestList, std::move(guests));
       }
-      guests->Append(base::WrapUnique(descriptor));
+      guests_weak->Append(base::WrapUnique(descriptor));
     }
   }
 
@@ -344,8 +345,7 @@ void AdbTargetsUIHandler::DeviceListChanged(
         kAdbDeviceIdFormat,
         device->serial().c_str());
     device_data->SetString(kTargetIdField, device_id);
-    base::ListValue* browser_list = new base::ListValue();
-    device_data->Set(kAdbBrowsersList, browser_list);
+    auto browser_list = base::MakeUnique<base::ListValue>();
 
     DevToolsAndroidBridge::RemoteBrowsers& browsers = device->browsers();
     for (DevToolsAndroidBridge::RemoteBrowsers::iterator bit =
@@ -365,9 +365,8 @@ void AdbTargetsUIHandler::DeviceListChanged(
       browser_data->SetString(kTargetIdField, browser_id);
       browser_data->SetString(kTargetSourceField, source_id());
 
-      base::ListValue* page_list = new base::ListValue();
+      auto page_list = base::MakeUnique<base::ListValue>();
       remote_browsers_[browser_id] = browser;
-            browser_data->Set(kAdbPagesList, page_list);
       for (const auto& page : browser->pages()) {
         scoped_refptr<DevToolsAgentHost> host = page->CreateTarget();
         std::unique_ptr<base::DictionaryValue> target_data = Serialize(host);
@@ -380,9 +379,11 @@ void AdbTargetsUIHandler::DeviceListChanged(
         targets_[host->GetId()] = host;
         page_list->Append(std::move(target_data));
       }
+      browser_data->Set(kAdbPagesList, std::move(page_list));
       browser_list->Append(std::move(browser_data));
     }
 
+    device_data->Set(kAdbBrowsersList, std::move(browser_list));
     device_list.Append(std::move(device_data));
   }
   SendSerializedTargets(device_list);
@@ -482,22 +483,22 @@ void PortForwardingStatusSerializer::PortStatusChanged(
   base::DictionaryValue result;
   for (ForwardingStatus::const_iterator sit = status.begin();
       sit != status.end(); ++sit) {
-    base::DictionaryValue* port_status_dict = new base::DictionaryValue();
+    auto port_status_dict = base::MakeUnique<base::DictionaryValue>();
     const PortStatusMap& port_status_map = sit->second;
     for (PortStatusMap::const_iterator it = port_status_map.begin();
          it != port_status_map.end(); ++it) {
       port_status_dict->SetInteger(base::IntToString(it->first), it->second);
     }
 
-    base::DictionaryValue* device_status_dict = new base::DictionaryValue();
-    device_status_dict->Set(kPortForwardingPorts, port_status_dict);
+    auto device_status_dict = base::MakeUnique<base::DictionaryValue>();
+    device_status_dict->Set(kPortForwardingPorts, std::move(port_status_dict));
     device_status_dict->SetString(kPortForwardingBrowserId,
                                   sit->first->GetId());
 
     std::string device_id = base::StringPrintf(
         kAdbDeviceIdFormat,
         sit->first->serial().c_str());
-    result.Set(device_id, device_status_dict);
+    result.Set(device_id, std::move(device_status_dict));
   }
   callback_.Run(result);
 }