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

Unified Diff: chrome/common/metrics/metrics_service_base.cc

Issue 11682003: Serialize/Deserialize support in HistogramBase (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years 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/common/metrics/metrics_service_base.cc
diff --git a/chrome/common/metrics/metrics_service_base.cc b/chrome/common/metrics/metrics_service_base.cc
index 84410c4f0dc595edf92ea4803a327798b35f6ff0..c0df1bdd2b86edc488926f1d5c08976d6b477366 100644
--- a/chrome/common/metrics/metrics_service_base.cc
+++ b/chrome/common/metrics/metrics_service_base.cc
@@ -38,7 +38,8 @@ void MetricsServiceBase::RecordCurrentHistograms() {
void MetricsServiceBase::RecordDelta(
const base::Histogram& histogram,
const base::HistogramSamples& snapshot) {
- log_manager_.current_log()->RecordHistogramDelta(histogram, snapshot);
+ log_manager_.current_log()->RecordHistogramDelta(histogram.histogram_name(),
Ilya Sherman 2012/12/29 00:17:30 This is a fine change, but why is it part of this
kaiwang 2013/01/08 00:51:40 In this CL, I changed most of the serializing rela
Ilya Sherman 2013/01/08 22:31:53 Ok, fair enough :)
+ snapshot);
}
void MetricsServiceBase::InconsistencyDetected(

Powered by Google App Engine
This is Rietveld 408576698