Add UMA to record the ratio of how many fetch events are not handled

content/browser/service_worker/service_worker_metrics.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/service_worker/service_worker_metrics.h"
 
 #include "base/metrics/histogram_macros.h"
 #include "base/metrics/user_metrics_action.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/content_browser_client.h"
@@ skipping 89 matching lines @@
   UMA_HISTOGRAM_ENUMERATION("ServiceWorker.ActivateEventStatus", status,
                             SERVICE_WORKER_ERROR_MAX_VALUE);
 }
 
 void ServiceWorkerMetrics::RecordInstallEventStatus(
     ServiceWorkerStatusCode status) {
   UMA_HISTOGRAM_ENUMERATION("ServiceWorker.InstallEventStatus", status,
                             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 function - you can merge this
with line 115.

[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 multiplication - i.e. * 100.

[kinuko, 2015/05/15 15:22:31]

Done.

+  UMA_HISTOGRAM_PERCENTAGE("ServiceWorker.UnhandledEventRatio",
+                           unhandled_ratio);
+}
+
 }  // namespace content