Add SampleMap and use it in SparseHistogram

base/metrics/sparse_histogram.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 "base/metrics/sparse_histogram.h"
 
 #include "base/metrics/statistics_recorder.h"
+#include "base/synchronization/lock.h"
 
+using std::map;
 using std::string;
 
 namespace base {
 
+typedef HistogramBase::Count Count;
+typedef HistogramBase::Sample Sample;
+
 // static
-HistogramBase* SparseHistogram::FactoryGet(const string& name,
-                                           int32 flags) {
+HistogramBase* SparseHistogram::FactoryGet(const string& name, int32 flags) {
   // TODO(kaiwang): Register and get SparseHistogram with StatisticsRecorder.
   HistogramBase* histogram = new SparseHistogram(name);
   histogram->SetFlags(flags);
   return histogram;
 }
 
 SparseHistogram::~SparseHistogram() {}
 
 void SparseHistogram::Add(Sample value) {
   base::AutoLock auto_lock(lock_);
-  samples_[value]++;
+  sample_counts_[value]++;
+  redundant_count_ += 1;
 }
 
-void SparseHistogram::SnapshotSample(std::map<Sample, Count>* samples) const {
+scoped_ptr<SampleMap> SparseHistogram::SnapshotSamples() const {
+  scoped_ptr<SampleMap> snapshot(new SampleMap());
+
   base::AutoLock auto_lock(lock_);
-  *samples = samples_;
+  for(map<Sample, Count>::const_iterator it = sample_counts_.begin();
+      it != sample_counts_.end();
+      ++it) {
+    snapshot->Accumulate(it->first, it->second);
+  }
+  snapshot->ResetRedundantCount(redundant_count_);
+  return snapshot.Pass();
 }
 
 void SparseHistogram::WriteHTMLGraph(string* output) const {
   // TODO(kaiwang): Implement.
 }
 
 void SparseHistogram::WriteAscii(string* output) const {
   // TODO(kaiwang): Implement.
 }
 
 SparseHistogram::SparseHistogram(const string& name)
-    : HistogramBase(name) {}
+    : HistogramBase(name),
+      redundant_count_(0) {}
 
 }  // namespace base