[UMA] Remove redundant bucket_count variable from base::Histogram.

base/metrics/statistics_recorder_unittest.cc

Index: base/metrics/statistics_recorder_unittest.cc
diff --git a/base/metrics/statistics_recorder_unittest.cc b/base/metrics/statistics_recorder_unittest.cc
index ea91e8e8e9b601e90b8658d429e789b26ffccefb..972bb637d1e792eb1f9777a62518b55af47cc8c6 100644
--- a/base/metrics/statistics_recorder_unittest.cc
+++ b/base/metrics/statistics_recorder_unittest.cc
@@ -36,7 +36,10 @@ class StatisticsRecorderTest : public testing::Test {
                              HistogramBase::Sample min,
                              HistogramBase::Sample max,
                              size_t bucket_count) {
-    return new Histogram(name, min, max, bucket_count, NULL);
+    BucketRanges* ranges = new BucketRanges(bucket_count + 1);
+    const BucketRanges* registered_ranges =
+        StatisticsRecorder::RegisterOrDeleteDuplicateRanges(ranges);

[jar (doing other things), 2013/06/20 16:41:18]

This tries to de-dup a set of ranges that are fully fleshed out.

It is not clear how we could have fleshed out the ranges if we haven't even
supplied the min and max values (that were given to us as arguments).

Perhaps you need to call:
 base::Histogram::InitializeBucketRanges(min, max, ranges)
after line 39 in onder to set these up.

[Ilya Sherman, 2013/06/21 04:03:05]

Done.

+    return new Histogram(name, min, max, registered_ranges);
   }
 
   void DeleteHistogram(HistogramBase* histogram) {