diff options
author | kaiwang@chromium.org <kaiwang@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-01 21:34:08 +0000 |
---|---|---|
committer | kaiwang@chromium.org <kaiwang@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-01 21:34:08 +0000 |
commit | 34d062327516ae2496646d4c4d644e0e12386a3d (patch) | |
tree | d92e5703447f10aea57b1bdcfea060600252be9b /chrome/common/metrics | |
parent | ba125e774ed899bd119bcd9abd47dab68857315b (diff) | |
download | chromium_src-34d062327516ae2496646d4c4d644e0e12386a3d.zip chromium_src-34d062327516ae2496646d4c4d644e0e12386a3d.tar.gz chromium_src-34d062327516ae2496646d4c4d644e0e12386a3d.tar.bz2 |
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
Diffstat (limited to 'chrome/common/metrics')
-rw-r--r-- | chrome/common/metrics/metrics_log_base.cc | 2 |
1 files changed, 1 insertions, 1 deletions
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[]. |