Remove ScopedVector in //component/metrics

chrome/browser/chromeos/external_metrics.cc

 // Copyright (c) 2012 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/chromeos/external_metrics.h"
 
 #include <stddef.h>
 
 #include <map>
 #include <string>
@@ skipping 114 matching lines @@
 
 void ExternalMetrics::RecordSparseHistogram(
     const metrics::MetricSample& sample) {
   CHECK_EQ(metrics::MetricSample::SPARSE_HISTOGRAM, sample.type());
   base::HistogramBase* counter = base::SparseHistogram::FactoryGet(
       sample.name(), base::HistogramBase::kUmaTargetedHistogramFlag);
   counter->Add(sample.sample());
 }
 
 int ExternalMetrics::CollectEvents() {
-  ScopedVector<metrics::MetricSample> samples;
+  std::vector<std::unique_ptr<metrics::MetricSample>> samples;

[Ilya Sherman, 2017/02/13 19:21:30]

Please #include <vector> and <memory> at the top of this file, since neither of
them is #included by the header.  Please go through for all of the files that
you've modified, and double-check whether these includes are needed there as
well.

[ke.he, 2017/02/14 04:24:14]

Done.

   metrics::SerializationUtils::ReadAndTruncateMetricsFromFile(uma_events_file_,
                                                               &samples);
 
-  for (ScopedVector<metrics::MetricSample>::iterator it = samples.begin();
-       it != samples.end();
-       ++it) {
+  for (auto it = samples.begin(); it != samples.end(); ++it) {
     const metrics::MetricSample& sample = **it;
 
     // Do not use the UMA_HISTOGRAM_... macros here.  They cache the Histogram
     // instance and thus only work if |sample.name()| is constant.
     switch (sample.type()) {
       case metrics::MetricSample::CRASH:
         RecordCrash(sample.name());
         break;
       case metrics::MetricSample::USER_ACTION:
         RecordAction(sample.name());
@@ skipping 20 matching lines @@
 
 void ExternalMetrics::ScheduleCollector() {
   bool result = BrowserThread::GetBlockingPool()->PostDelayedWorkerTask(
       FROM_HERE,
       base::Bind(&chromeos::ExternalMetrics::CollectEventsAndReschedule, this),
       base::TimeDelta::FromSeconds(kExternalMetricsCollectionIntervalSeconds));
   DCHECK(result);
 }
 
 }  // namespace chromeos