Change base::Value::ListStorage to std::vector<base::Value>

chromeos/printing/ppd_provider.cc

Index: chromeos/printing/ppd_provider.cc
diff --git a/chromeos/printing/ppd_provider.cc b/chromeos/printing/ppd_provider.cc
index a6a08c59ec79ca9a5e0938bc4afe1337b072139b..37130fc02b61a536952ace5bc62ef4ae09f9b6a5 100644
--- a/chromeos/printing/ppd_provider.cc
+++ b/chromeos/printing/ppd_provider.cc
@@ -431,11 +431,11 @@ class PpdProviderImpl : public PpdProvider, public net::URLFetcherDelegate {
     // This should just be a simple list of locale strings.
     std::vector<std::string> available_locales;
     bool found_en = false;
-    for (const std::unique_ptr<base::Value>& entry : *top_list) {
+    for (const base::Value& entry : *top_list) {
       std::string tmp;
       // Locales should have at *least* a two-character country code.  100 is an
       // arbitrary upper bound for length to protect against extreme bogosity.
-      if (!entry->GetAsString(&tmp) || tmp.size() < 2 || tmp.size() > 100) {
+      if (!entry.GetAsString(&tmp) || tmp.size() < 2 || tmp.size() > 100) {
         FailQueuedMetadataResolutions(PpdProvider::INTERNAL_ERROR);
         return;
       }
@@ -603,10 +603,10 @@ class PpdProviderImpl : public PpdProvider, public net::URLFetcherDelegate {
         result = PpdProvider::NOT_FOUND;
         for (const auto& entry : *top_list) {
           int device_id;
-          base::ListValue* sub_list;
+          const base::ListValue* sub_list;
 
           // Each entry should be a size-2 list with an integer and a string.
-          if (!entry->GetAsList(&sub_list) || sub_list->GetSize() != 2 ||
+          if (!entry.GetAsList(&sub_list) || sub_list->GetSize() != 2 ||
               !sub_list->GetInteger(0, &device_id) ||
               !sub_list->GetString(1, &contents) || device_id < 0 ||
               device_id > 0xffff) {
@@ -791,9 +791,9 @@ class PpdProviderImpl : public PpdProvider, public net::URLFetcherDelegate {
       return PpdProvider::INTERNAL_ERROR;
     }
     for (const auto& entry : *top_list) {
-      base::ListValue* sub_list;
+      const base::ListValue* sub_list;
       contents->push_back({});
-      if (!entry->GetAsList(&sub_list) || sub_list->GetSize() != 2 ||
+      if (!entry.GetAsList(&sub_list) || sub_list->GetSize() != 2 ||
           !sub_list->GetString(0, &contents->back().first) ||
           !sub_list->GetString(1, &contents->back().second)) {
         contents->clear();