Android - UMA for notification click latency via NotificationJobService

chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationConstants.java

Index: chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationConstants.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationConstants.java b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationConstants.java
index 84b5fff48401703942a2d45af3a1f0d5c8257a4e..09fc5cef464d7cd81b917bcb878a56e9ee475b6d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationConstants.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/notifications/NotificationConstants.java
@@ -40,6 +40,9 @@ public class NotificationConstants {
     static final String EXTRA_NOTIFICATION_REPLY = "notification_reply";
     static final String EXTRA_NOTIFICATION_ACTION = "notification_action";
 
+    static final String NOTIFICATION_JOB_SCHEDULED_TIME_MS = "notification_job_scheduled_time_ms";
+    static final String NOTIFICATION_JOB_STARTED_TIME_MS = "notification_job_started_time_ms";

[Peter Beverloo, 2017/03/22 17:25:24]

nit: these are extras, why not prefix them as such? I'd drop NOTIFICATION_ if
it's getting too long, that's already clear from the class' name.

[awdf, 2017/03/22 18:14:51]

Done.

+
     /**
      * Unique identifier for a single sync notification. Since the notification ID is reused,
      * old notifications will be overwritten.