Push API: UMA logs of whether SWs show notifications when required

content/public/common/push_messaging_status.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_
 #define CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_
 
 namespace content {
 
 // Push registration success / error codes for internal use & reporting in UMA.
@@ skipping 90 matching lines @@
   // The message was delivered, but the Service Worker passed a Promise to
   // event.waitUntil that got rejected.
   PUSH_DELIVERY_STATUS_EVENT_WAITUNTIL_REJECTED,
 
   // When making changes, update PUSH_DELIVERY_STATUS_LAST below.
 
   // Used for IPC message range checks.
   PUSH_DELIVERY_STATUS_LAST = PUSH_DELIVERY_STATUS_EVENT_WAITUNTIL_REJECTED
 };
 
+// Push message user visible tracking for reporting in UMA.
+enum PushUserVisibleStatus {
+  // A notification was required and one (or more) were shown.
+  PUSH_USER_VISIBLE_STATUS_REQUIRED_AND_SHOWN = 0,
+
+  // A notification was not required, but one (or more) were shown anyway.
+  PUSH_USER_VISIBLE_STATUS_NOT_REQUIRED_BUT_SHOWN = 1,
+
+  // A notification was not required and none were shown.
+  PUSH_USER_VISIBLE_STATUS_NOT_REQUIRED_AND_NOT_SHOWN = 2,
+
+  // A notification was required, but none were shown
+  PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_USED_GRACE = 3,
+
+  // A notification was required, but none were shown
+  PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_GRACE_EXCEEDED = 4,
+

[Miguel Garcia, 2015/02/13 11:26:25]

can we have a explicit UMA for "The default notification was shown". Maybe not
in this enum but as an independent counter.

[johnme, 2015/02/13 18:36:26]

Yes, that's precisely what
PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_GRACE_EXCEEDED does. My comment
was unclear; I've updated the comment.

+  // NOTE: Do not renumber these as that would confuse interpretation of
+  // previously logged data. When making changes, also update the enum list
+  // in tools/metrics/histograms/histograms.xml to keep it in sync, and
+  // update PUSH_USER_VISIBLE_STATUS_LAST below.
+
+  // Used for IPC message range checks.
+  PUSH_USER_VISIBLE_STATUS_LAST =
+      PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_GRACE_EXCEEDED
+};
+
 const char* PushRegistrationStatusToString(PushRegistrationStatus status);
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_COMMON_PUSH_MESSAGING_STATUS_H_