UMA for Notification action buttons

third_party/WebKit/Source/modules/notifications/ServiceWorkerRegistrationNotifications.cpp

Index: third_party/WebKit/Source/modules/notifications/ServiceWorkerRegistrationNotifications.cpp
diff --git a/third_party/WebKit/Source/modules/notifications/ServiceWorkerRegistrationNotifications.cpp b/third_party/WebKit/Source/modules/notifications/ServiceWorkerRegistrationNotifications.cpp
index d916a0503ac76eaef6352b86f9d4853b8c0167e6..c6aad2d825b4d59b4411accde84153bd107c1a78 100644
--- a/third_party/WebKit/Source/modules/notifications/ServiceWorkerRegistrationNotifications.cpp
+++ b/third_party/WebKit/Source/modules/notifications/ServiceWorkerRegistrationNotifications.cpp
@@ -60,6 +60,9 @@ ScriptPromise ServiceWorkerRegistrationNotifications::showNotification(ScriptSta
     if (exceptionState.hadException())
         return exceptionState.reject(scriptState);
 
+    // Log number of actions developer provided, even if it's more than we currently support.

[Alexei Svitkine (slow), 2015/11/23 16:19:19]

Nit: Expand comment to mention you're intentionally using a linear histogram
here and where the magic number '17' came from.

[johnme, 2015/11/24 11:24:12]

Done.

+    Platform::current()->histogramEnumeration("Notifications.PersistentNotificationActionCount", options.actions().size(), 17);
+
     ScriptPromiseResolver* resolver = ScriptPromiseResolver::create(scriptState);
     ScriptPromise promise = resolver->promise();