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

chrome/browser/metrics/plugin_metrics_provider.cc

 // Copyright 2014 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/metrics/plugin_metrics_provider.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
@@ skipping 126 matching lines @@
     metrics::SystemProfileProto* system_profile_proto) {
   RecordCurrentStateIfPending();
   const base::ListValue* plugin_stats_list = local_state_->GetList(
       prefs::kStabilityPluginStats);
   if (!plugin_stats_list)
     return;
 
   metrics::SystemProfileProto::Stability* stability =
       system_profile_proto->mutable_stability();
   for (const auto& value : *plugin_stats_list) {
-    const base::DictionaryValue* plugin_dict;
-    if (!value.GetAsDictionary(&plugin_dict)) {
+    base::DictionaryValue* plugin_dict;
+    if (!value->GetAsDictionary(&plugin_dict)) {
       NOTREACHED();
       continue;
     }
 
     // Note that this search is potentially a quadratic operation, but given the
     // low number of plugins installed on a "reasonable" setup, this should be
     // fine.
     // TODO(isherman): Verify that this does not show up as a hotspot in
     // profiler runs.
     const metrics::SystemProfileProto::Plugin* system_profile_plugin = NULL;
@@ skipping 44 matching lines @@
   local_state_->ClearPref(prefs::kStabilityPluginStats);
 }
 
 // Saves plugin-related updates from the in-object buffer to Local State
 // for retrieval next time we send a Profile log (generally next launch).
 void PluginMetricsProvider::RecordCurrentState() {
   ListPrefUpdate update(local_state_, prefs::kStabilityPluginStats);
   base::ListValue* plugins = update.Get();
   DCHECK(plugins);
 
-  for (auto& value : *plugins) {
+  for (const auto& value : *plugins) {
     base::DictionaryValue* plugin_dict;
-    if (!value.GetAsDictionary(&plugin_dict)) {
+    if (!value->GetAsDictionary(&plugin_dict)) {
       NOTREACHED();
       continue;
     }
 
     base::string16 plugin_name;
     plugin_dict->GetString(prefs::kStabilityPluginName, &plugin_name);
     if (plugin_name.empty()) {
       NOTREACHED();
       continue;
     }
@@ skipping 151 matching lines @@
 }
 
 bool PluginMetricsProvider::RecordCurrentStateIfPending() {
   if (!weak_ptr_factory_.HasWeakPtrs())
     return false;
 
   weak_ptr_factory_.InvalidateWeakPtrs();
   RecordCurrentState();
   return true;
 }