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

base/metrics/histogram_unittest.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.
 
 // Test of Histogram class
 
 #include <climits>
 #include <algorithm>
 #include <vector>
 
@@ skipping 72 matching lines @@
       "DuplicatedHistogram", 1, 101, 102, HistogramBase::kNoFlags);
 
   scoped_ptr<HistogramSamples> samples = histogram->SnapshotSamples();
   EXPECT_EQ(2, samples->TotalCount());
   EXPECT_EQ(2, samples->GetCount(10));
 }
 
 TEST_F(HistogramTest, ExponentialRangesTest) {
   // Check that we got a nice exponential when there was enough rooom.
   BucketRanges ranges(9);
-  Histogram::InitializeBucketRanges(1, 64, 8, &ranges);
+  Histogram::InitializeBucketRanges(1, 64, &ranges);
   EXPECT_EQ(0, ranges.range(0));
   int power_of_2 = 1;
   for (int i = 1; i < 8; i++) {
     EXPECT_EQ(power_of_2, ranges.range(i));
     power_of_2 *= 2;
   }
   EXPECT_EQ(HistogramBase::kSampleType_MAX, ranges.range(8));
 
   // Check the corresponding Histogram will use the correct ranges.
   Histogram* histogram = static_cast<Histogram*>(
       Histogram::FactoryGet("Histogram", 1, 64, 8, HistogramBase::kNoFlags));
   EXPECT_TRUE(ranges.Equals(histogram->bucket_ranges()));
 
   // When bucket count is limited, exponential ranges will partially look like
   // linear.
   BucketRanges ranges2(16);
-  Histogram::InitializeBucketRanges(1, 32, 15, &ranges2);
+  Histogram::InitializeBucketRanges(1, 32, &ranges2);
 
   EXPECT_EQ(0, ranges2.range(0));
   EXPECT_EQ(1, ranges2.range(1));
   EXPECT_EQ(2, ranges2.range(2));
   EXPECT_EQ(3, ranges2.range(3));
   EXPECT_EQ(4, ranges2.range(4));
   EXPECT_EQ(5, ranges2.range(5));
   EXPECT_EQ(6, ranges2.range(6));
   EXPECT_EQ(7, ranges2.range(7));
   EXPECT_EQ(9, ranges2.range(8));
   EXPECT_EQ(11, ranges2.range(9));
   EXPECT_EQ(14, ranges2.range(10));
   EXPECT_EQ(17, ranges2.range(11));
   EXPECT_EQ(21, ranges2.range(12));
   EXPECT_EQ(26, ranges2.range(13));
   EXPECT_EQ(32, ranges2.range(14));
   EXPECT_EQ(HistogramBase::kSampleType_MAX, ranges2.range(15));
 
   // Check the corresponding Histogram will use the correct ranges.
   Histogram* histogram2 = static_cast<Histogram*>(
       Histogram::FactoryGet("Histogram2", 1, 32, 15, HistogramBase::kNoFlags));
   EXPECT_TRUE(ranges2.Equals(histogram2->bucket_ranges()));
 }
 
 TEST_F(HistogramTest, LinearRangesTest) {
   BucketRanges ranges(9);
-  LinearHistogram::InitializeBucketRanges(1, 7, 8, &ranges);
+  LinearHistogram::InitializeBucketRanges(1, 7, &ranges);
   // Gets a nice linear set of bucket ranges.
   for (int i = 0; i < 8; i++)
     EXPECT_EQ(i, ranges.range(i));
   EXPECT_EQ(HistogramBase::kSampleType_MAX, ranges.range(8));
 
   // The correspoding LinearHistogram should use the correct ranges.
   Histogram* histogram = static_cast<Histogram*>(
       LinearHistogram::FactoryGet("Linear", 1, 7, 8, HistogramBase::kNoFlags));
   EXPECT_TRUE(ranges.Equals(histogram->bucket_ranges()));
 
   // Linear ranges are not divisible.
   BucketRanges ranges2(6);
-  LinearHistogram::InitializeBucketRanges(1, 6, 5, &ranges2);
+  LinearHistogram::InitializeBucketRanges(1, 6, &ranges2);
   EXPECT_EQ(0, ranges2.range(0));
   EXPECT_EQ(1, ranges2.range(1));
   EXPECT_EQ(3, ranges2.range(2));
   EXPECT_EQ(4, ranges2.range(3));
   EXPECT_EQ(6, ranges2.range(4));
   EXPECT_EQ(HistogramBase::kSampleType_MAX, ranges2.range(5));
   // The correspoding LinearHistogram should use the correct ranges.
   Histogram* histogram2 = static_cast<Histogram*>(
       LinearHistogram::FactoryGet("Linear2", 1, 6, 5, HistogramBase::kNoFlags));
   EXPECT_TRUE(ranges2.Equals(histogram2->bucket_ranges()));
@@ skipping 323 matching lines @@
   // CustomHistogram needs at least 1 valid range.
   custom_ranges.clear();
   custom_ranges.push_back(0);
   EXPECT_DEATH(CustomHistogram::FactoryGet("BadRangesCustom3", custom_ranges,
                                            HistogramBase::kNoFlags),
                "");
 }
 #endif
 
 }  // namespace base