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 {

[no sievers, 2015/02/17 20:26:56]

Hmm ok, it was already like this, but 
http://www.chromium.org/developers/content-module/content-api says that each
enum should be in its own file.

+  // 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. Fortunately, the site has
+  // been well behaved recently so it was glossed over.
+  PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_USED_GRACE = 3,
+
+  // A notification was required, but none were shown. Unfortunately, the site
+  // has run out of grace, so we had to show the user a generic notification.
+  PUSH_USER_VISIBLE_STATUS_REQUIRED_BUT_NOT_SHOWN_GRACE_EXCEEDED = 4,
+
+  // 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_