Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(244)

Unified Diff: chrome/browser/protector/default_search_provider_change_browsertest.cc

Issue 10829466: SampleSet -> HistogramSamples (will be reused by SparseHistogram) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/protector/default_search_provider_change_browsertest.cc
===================================================================
--- chrome/browser/protector/default_search_provider_change_browsertest.cc (revision 152603)
+++ chrome/browser/protector/default_search_provider_change_browsertest.cc (working copy)
@@ -130,10 +130,10 @@
base::Histogram::Count count) {
base::Histogram* histogram = base::StatisticsRecorder::FindHistogram(name);
EXPECT_TRUE(histogram != NULL);
- base::Histogram::SampleSet sample;
- histogram->SnapshotSample(&sample);
- EXPECT_EQ(count, sample.counts(bucket)) <<
- "Invalid " << name << " value for bucket " << bucket;
+ scoped_ptr<base::BucketHistogramSamples> samples =
+ histogram->SnapshotSamples();
+ EXPECT_EQ(count, samples->GetCountFromBucketIndex(bucket))
+ << "Invalid " << name << " value for bucket " << bucket;
}
protected:

Powered by Google App Engine
This is Rietveld 408576698