Chromium Code Reviews| 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 |