Index: content/browser/service_worker/service_worker_metrics.cc |
diff --git a/content/browser/service_worker/service_worker_metrics.cc b/content/browser/service_worker/service_worker_metrics.cc |
index dc3c157ab2cc9310699590c828902b3d442c97b4..4602a4f48239fa9e19acda6db301dd72db63ec0d 100644 |
--- a/content/browser/service_worker/service_worker_metrics.cc |
+++ b/content/browser/service_worker/service_worker_metrics.cc |
@@ -107,4 +107,14 @@ void ServiceWorkerMetrics::RecordInstallEventStatus( |
SERVICE_WORKER_ERROR_MAX_VALUE); |
} |
+void ServiceWorkerMetrics::RecordEventStatus(size_t fired_events, |
+ size_t handled_events) { |
+ int unhandled_ratio = 0; |
Alexei Svitkine (slow)
2015/05/15 14:35:57
Nit: No need to declare this at the top of the fun
kinuko
2015/05/15 15:22:31
Yeah, done.
|
+ if (!fired_events) |
+ return; |
+ unhandled_ratio = (fired_events - handled_events) * 100.0 / fired_events; |
Alexei Svitkine (slow)
2015/05/15 14:35:57
Nit: No need for 100.0 - can just do integer multi
kinuko
2015/05/15 15:22:31
Done.
|
+ UMA_HISTOGRAM_PERCENTAGE("ServiceWorker.UnhandledEventRatio", |
+ unhandled_ratio); |
+} |
+ |
} // namespace content |