From 34d062327516ae2496646d4c4d644e0e12386a3d Mon Sep 17 00:00:00 2001 From: "kaiwang@chromium.org" Date: Wed, 1 Aug 2012 21:34:08 +0000 Subject: This is a major refactor of Histogram related code: 1. Remove duplicated code from histogram.h/.cc, including validating related code and BucketRanges related. 2. Constness of BucketRanges from Histograms, to prevent accidentally modification and provide a simpler interface. 3. Add/move tests. Review URL: https://chromiumcodereview.appspot.com/10834011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149495 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/common/metrics/metrics_log_base.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'chrome/common/metrics') diff --git a/chrome/common/metrics/metrics_log_base.cc b/chrome/common/metrics/metrics_log_base.cc index 8617c92..e16b46d 100644 --- a/chrome/common/metrics/metrics_log_base.cc +++ b/chrome/common/metrics/metrics_log_base.cc @@ -452,7 +452,7 @@ void MetricsLogBase::RecordHistogramDelta( const Histogram::SampleSet& snapshot) { DCHECK(!locked_); DCHECK_NE(0, snapshot.TotalCount()); - snapshot.CheckSize(histogram); + DCHECK_EQ(histogram.bucket_count(), snapshot.size()); // We will ignore the MAX_INT/infinite value in the last element of range[]. -- cgit v1.1