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

chrome/test/chromedriver/performance_logger.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/test/chromedriver/performance_logger.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 172 matching lines @@
     trace_buffering_ = false;
   } else if (method == "Tracing.dataCollected") {
     // The Tracing.dataCollected event contains a list of trace events.
     // Add each one as an individual log entry of method Tracing.dataCollected.
     const base::ListValue* traces;
     if (!params.GetList("value", &traces)) {
       return Status(kUnknownError,
                     "received DevTools trace data in unexpected format");
     }
     for (const auto& trace : *traces) {
-      const base::DictionaryValue* event_dict;
-      if (!trace.GetAsDictionary(&event_dict))
+      base::DictionaryValue* event_dict;
+      if (!trace->GetAsDictionary(&event_dict))
         return Status(kUnknownError, "trace event must be a dictionary");
       AddLogEntry(client->GetId(), "Tracing.dataCollected", *event_dict);
     }
   } else if (method == "Tracing.bufferUsage") {
     // 'value' will be between 0-1 and represents how full the DevTools trace
     // buffer is. If the buffer is full, warn the user.
     double buffer_usage = 0;
     if (!params.GetDouble("percentFull", &buffer_usage)) {
       // Tracing.bufferUsage event will occur once per second, and it really
       // only serves as a warning, so if we can't reliably tell whether the
@@ skipping 82 matching lines @@
   if (status.IsError())
     return status;
 
   return StartTrace();
 }
 
 Status PerformanceLogger::IsTraceDone(bool* trace_done) const {
   *trace_done = !trace_buffering_;
   return Status(kOk);
 }