diff options
| author | isherman@chromium.org <isherman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-27 14:38:45 +0000 |
|---|---|---|
| committer | isherman@chromium.org <isherman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-06-27 14:38:45 +0000 |
| commit | 15ce384fde9be34c54a15fcf784c229ce3a236e6 (patch) | |
| tree | 6f816a140cf5a313ecd533380f6d2025feac4027 /base/metrics/histogram_unittest.cc | |
| parent | 2e1e3a21fa81f5f1b920b81174e4d4a4ce889156 (diff) | |
| download | chromium_src-15ce384fde9be34c54a15fcf784c229ce3a236e6.zip chromium_src-15ce384fde9be34c54a15fcf784c229ce3a236e6.tar.gz chromium_src-15ce384fde9be34c54a15fcf784c229ce3a236e6.tar.bz2 | |
[UMA] Remove redundant bucket_count variable from base::Histogram.
BUG=248623
TEST=none
R=jar@chromium.org
Review URL: https://chromiumcodereview.appspot.com/17451016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@208917 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/metrics/histogram_unittest.cc')
| -rw-r--r-- | base/metrics/histogram_unittest.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/base/metrics/histogram_unittest.cc b/base/metrics/histogram_unittest.cc index b5081b6..6ff7d00 100644 --- a/base/metrics/histogram_unittest.cc +++ b/base/metrics/histogram_unittest.cc @@ -90,7 +90,7 @@ TEST_F(HistogramTest, NameMatchTest) { 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++) { @@ -107,7 +107,7 @@ TEST_F(HistogramTest, ExponentialRangesTest) { // 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)); @@ -134,7 +134,7 @@ TEST_F(HistogramTest, ExponentialRangesTest) { 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)); @@ -147,7 +147,7 @@ TEST_F(HistogramTest, LinearRangesTest) { // 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)); |