tools/metrics/histograms/histograms.xml

Index: tools/metrics/histograms/histograms.xml
diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml
index f650bb60e7a58256092200905699f15e67b430fb..c852c1a06e4089d6726f3f4000d9a5c688dd8a29 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -56058,6 +56058,32 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <summary>URLs for which Chrome creates WebUIControllers.</summary>
 </histogram>
 
+<histogram name="WebUsb.ChooserClosed" enum="WebUsbChooserClosed">
+  <owner>reillyg@chromium.org</owner>
+  <owner>juncai@chromium.org</owner>
+  <summary>
+    Records the reason why the WebUSB device chooser was closed.
+  </summary>
+</histogram>
+
+<histogram name="WebUsb.NotificationClosed" enum="WebUsbNotificationClosed">
+  <owner>reillyg@chromium.org</owner>
+  <owner>juncai@chromium.org</owner>
+  <summary>
+    Records the reason why the &quot;WebUSB device detected&quot; notification
+    was dismissed.
+  </summary>
+</histogram>
+
+<histogram name="WebUsb.PermissionRevoked" enum="WebUsbPermissionRevoked">
+  <owner>reillyg@chromium.org</owner>
+  <owner>juncai@chromium.org</owner>
+  <summary>
+    Records when the user revokes permission for an origin to connect to a USB
+    device using the WebUSB API.
+  </summary>
+</histogram>
+
 <histogram name="Win8.PageLoad" enum="Win8PageLoadType">
   <owner>zturner@chromium.org</owner>
   <summary>
@@ -80725,6 +80751,24 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="2114840772" label="chrome://drive-internals/"/>
 </enum>
 
+<enum name="WebUsbChooserClosed" type="int">
+  <int value="0" label="User cancelled"/>
+  <int value="1" label="User cancelled, no devices available"/>
+  <int value="2" label="Permission granted"/>
+  <int value="3" label="Ephemeral permission granted"/>
+</enum>
+
+<enum name="WebUsbNotificationClosed" type="int">
+  <int value="0" label="Closed"/>
+  <int value="1" label="Closed by user"/>

[asargent_no_longer_on_chrome, 2016/01/21 18:54:15]

It might be good to provide an explanation for the distinction between "Closed"
and "Closed by user" as it's not immediately obvious from their names. I can't
recall off the top of my head if enum's support explanatory comments, but if
they don't you could just add something to the <summary> block for
WebUsb.NotificationClosed

+  <int value="2" label="Clicked"/>
+</enum>
+
+<enum name="WebUsbPermissionRevoked" type="int">
+  <int value="0" label="Revoked permission"/>
+  <int value="1" label="Revoked ephemeral permission"/>
+</enum>
+
 <enum name="WiFiApMode" type="int">
   <int value="0" label="Unknown"/>
   <int value="1" label="Managed"/>