Chromium Code Reviews

Unified Diff: components/metrics/metrics_service.cc

Issue 1537743006: Persist setup metrics and have Chrome report them during UMA upload. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@shared-histograms
Patch Set: moved allocator create/destroy to be beside snapshot (simpler code) Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: components/metrics/metrics_service.cc
diff --git a/components/metrics/metrics_service.cc b/components/metrics/metrics_service.cc
index 0b84dffbf361f48fe5554ade4e2139c78a666660..46dee9adcb995c34b761b203ed970e5e84cc45a7 100644
--- a/components/metrics/metrics_service.cc
+++ b/components/metrics/metrics_service.cc
@@ -1107,6 +1107,12 @@ void MetricsService::RecordCurrentEnvironment(MetricsLog* log) {
void MetricsService::RecordCurrentHistograms() {
DCHECK(log_manager_.current_log());
+ histogram_snapshot_manager_.StartDeltas();
+ for (size_t i = 0; i < metrics_providers_.size(); ++i) {
+ metrics_providers_[i]->RecordHistogramSnapshots(
+ &histogram_snapshot_manager_);
+ }
+ histogram_snapshot_manager_.FinishDeltas();
}
void MetricsService::RecordCurrentStabilityHistograms() {
« components/metrics/file_metrics_provider.cc ('K') | « components/metrics/metrics_provider.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine