Update ServiceWorker event handled status histogram

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"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/common/content_client.h"
 
 namespace content {
 
 namespace {
 
 void RecordURLMetricOnUI(const GURL& url) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   GetContentClient()->browser()->RecordURLMetric(
       "ServiceWorker.ControlledPageUrl", url);
 }
 
+enum EventHandledStatus {
+  EVENT_HANDLED,
+  EVENT_NOT_HANDLED,
+  EVENT_HANDLED_STATUS_MAX,
+};
+
+enum PerWorkerEventHandledStatus {
+  NO_EVENTS_HANDLED,
+  SOME_EVENTS_HANDLED,
+  ALL_EVENTS_HANDLED,
+  PER_WORKER_EVENT_HANDLED_STATUS_MAX,
+};
+
 }  // namespace
 
 void ServiceWorkerMetrics::CountInitDiskCacheResult(bool result) {
   UMA_HISTOGRAM_BOOLEAN("ServiceWorker.DiskCache.InitResult", result);
 }
 
 void ServiceWorkerMetrics::CountReadResponseResult(
     ServiceWorkerMetrics::ReadResponseResult result) {
   UMA_HISTOGRAM_ENUMERATION("ServiceWorker.DiskCache.ReadResponseResult",
                             result, NUM_READ_RESPONSE_RESULT_TYPES);
@@ skipping 66 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) {
+void ServiceWorkerMetrics::RecordPerWorkerEventHandledStatus(
+    size_t fired_events,
+    size_t handled_events) {
   if (!fired_events)
     return;
-  int unhandled_ratio = (fired_events - handled_events) * 100 / fired_events;
-  UMA_HISTOGRAM_PERCENTAGE("ServiceWorker.UnhandledEventRatio",
-                           unhandled_ratio);
+  PerWorkerEventHandledStatus status = SOME_EVENTS_HANDLED;
+  if (fired_events == handled_events)
+    status = ALL_EVENTS_HANDLED;
+  else if (handled_events == 0)
+    status = NO_EVENTS_HANDLED;
+  UMA_HISTOGRAM_ENUMERATION("ServiceWorker.EventHandledStatus_PerWorker",
+                            status, PER_WORKER_EVENT_HANDLED_STATUS_MAX);
+}
+
+void ServiceWorkerMetrics::RecordEventHandledStatus(bool handled) {
+  if (handled) {
+    UMA_HISTOGRAM_ENUMERATION("ServiceWorker.EventHandledStatus", EVENT_HANDLED,
+                              EVENT_HANDLED_STATUS_MAX);
+  } else {
+    UMA_HISTOGRAM_ENUMERATION("ServiceWorker.EventHandledStatus",
+                              EVENT_NOT_HANDLED, EVENT_HANDLED_STATUS_MAX);
+  }
 }
 
 }  // namespace content