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;
+  if (handled_events > 0u)

[nhiroki, 2015/05/15 02:41:25]

This should be |fired_events| to avoid divining by 0.

[kinuko, 2015/05/15 04:42:23]

Done. I'm extremely careless these days.

+    unhandled_ratio = (fired_events - handled_events) * 100.0 / fired_events;
+  UMA_HISTOGRAM_PERCENTAGE("ServiceWorker.UnhandledEventRatio",
+                           unhandled_ratio);

[nhiroki, 2015/05/15 02:41:25]

Do we need to record in the case where |fired_events| is 0? This could record
noise because a version failed to start can go through here.

[kinuko, 2015/05/15 04:42:23]

Done.

+}
+
 }  // namespace content