Use std::unique_ptr for base::DictionaryValue and base::ListValue's internal store.

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 163 matching lines @@
   if (method == "Tracing.tracingComplete") {
     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 (base::Value* const trace : *traces) {
+    for (const auto& trace : *traces) {
       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("value", &buffer_usage)) {
@@ skipping 78 matching lines @@
   if (status.IsError())
     return status;
 
   return StartTrace();
 }
 
 Status PerformanceLogger::IsTraceDone(bool* trace_done) const {
   *trace_done = !trace_buffering_;
   return Status(kOk);
 }