SampleSet -> HistogramSamples (will be reused by SparseHistogram)

net/disk_cache/stats_histogram.h

Index: net/disk_cache/stats_histogram.h
===================================================================
--- net/disk_cache/stats_histogram.h	(revision 155400)
+++ net/disk_cache/stats_histogram.h	(working copy)
@@ -7,8 +7,15 @@
 
 #include <string>
 
+#include "base/memory/scoped_ptr.h"
 #include "base/metrics/histogram.h"
 
+namespace base {
+class BucketRanges;
+class SampleVector;
+class HistogramSamples;

[jar (doing other things), 2012/09/21 19:57:41]

tiny nit: probably alphabetize.

[kaiwang, 2012/09/21 19:59:35]

Done.

+}  // namespace base
+
 namespace disk_cache {
 
 class Stats;
@@ -22,34 +29,25 @@
 //
 class StatsHistogram : public base::Histogram {
  public:
-  class StatsSamples : public SampleSet {
-   public:
-    Counts* GetCounts() {
-      return &counts_;
-    }
-  };
-
   StatsHistogram(const std::string& name,
                  Sample minimum,
                  Sample maximum,
-                 size_t bucket_count)
-      : Histogram(name, minimum, maximum, bucket_count, NULL), init_(false) {}
+                 size_t bucket_count,
+                 base::BucketRanges* ranges,
+                 const Stats* stats);
   virtual ~StatsHistogram();
 
-  static StatsHistogram* FactoryGet(const std::string& name);
+  static void InitializeBucketRanges(const Stats* stats,
+                                     base::BucketRanges* ranges);
+  static StatsHistogram* FactoryGet(const std::string& name,
+                                    const Stats* stats);
 
-  // We'll be reporting data from the given set of cache stats.
-  bool Init(const Stats* stats);
-
-  virtual Sample ranges(size_t i) const OVERRIDE;
-  virtual size_t bucket_count() const OVERRIDE;
-  virtual void SnapshotSample(SampleSet* sample) const OVERRIDE;
+  virtual scoped_ptr<base::SampleVector> SnapshotSamples() const OVERRIDE;
   virtual Inconsistencies FindCorruption(
-      const SampleSet& snapshot) const OVERRIDE;
+      const base::HistogramSamples& samples) const OVERRIDE;
 
  private:
-  bool init_;
-  static const Stats* stats_;
+  const Stats* stats_;
   DISALLOW_COPY_AND_ASSIGN(StatsHistogram);
 };