GetTotalCountsForPrefix method for the histogram testing matches not only prefixes. |
|||
Issue description
GetTotalCountsForPrefix("MetricA") will match not only "MetricA.B.C", but also "A.MetricA.B.C
,
Jan 11 2017
,
Jan 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/63e22b6fcb629ebccc0f257de04bc8e7f4ef247c commit 63e22b6fcb629ebccc0f257de04bc8e7f4ef247c Author: pkalinnikov <pkalinnikov@chromium.org> Date: Fri Jan 13 18:30:38 2017 Match only prefixes of histogram names. Fix the bug that HistogramTester::GetTotalCountsForPrefix returns histograms which contain the query string as substrings, not as prefixes. Additionally, omit explicit base:: namespace here and there, because the files are already in this namespace. BUG= 659977 Review-Url: https://codereview.chromium.org/2628843002 Cr-Commit-Position: refs/heads/master@{#443620} [modify] https://crrev.com/63e22b6fcb629ebccc0f257de04bc8e7f4ef247c/base/test/histogram_tester.cc [modify] https://crrev.com/63e22b6fcb629ebccc0f257de04bc8e7f4ef247c/base/test/histogram_tester.h [modify] https://crrev.com/63e22b6fcb629ebccc0f257de04bc8e7f4ef247c/base/test/histogram_tester_unittest.cc
,
Jan 13 2017
|
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, Oct 28 2016