bluetooth: Add histograms and logging for requestDevice()

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 3f3fcf4d488573a99a212843a4a0e119b6e77131..d997870bfada9a156fb69b2955006d384b74b697 100644
--- a/tools/metrics/histograms/histograms.xml
+++ b/tools/metrics/histograms/histograms.xml
@@ -2739,6 +2739,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
 
 <histogram name="Bluetooth.RequestDevice.Outcome"
     enum="BluetoothRequestDeviceOutcome">
+  <obsolete>
+    Deprecated 08/2015 and replaced by Bluetooth.Web.RequestDevice.Outcome.
+  </obsolete>
   <owner>jyasskin@chromium.org</owner>
   <owner>ortuno@chromium.org</owner>
   <owner>scheib@chromium.org</owner>
@@ -2758,6 +2761,89 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   </summary>
 </histogram>
 
+<histogram name="Bluetooth.Web.RequestDevice.Filters.Count" units="filters">
+  <owner>jyasskin@chromium.org</owner>
+  <owner>ortuno@chromium.org</owner>
+  <owner>scheib@chromium.org</owner>
+  <summary>
+    Records how many filters are used when scanning. This results will help us
+    better understand the uses of the API and make changes according to
+    developers' behavior.
+  </summary>
+</histogram>
+
+<histogram name="Bluetooth.Web.RequestDevice.Filters.Services"
+    enum="GATTServiceHash">
+  <owner>jyasskin@chromium.org</owner>
+  <owner>ortuno@chromium.org</owner>
+  <owner>scheib@chromium.org</owner>
+  <summary>
+    Records a 31-bit hash of the GATT Service UUIDs used when scanning. This
+    will help us know if we should build an API for specific services rather
+    than a general Bluetooth API.
+  </summary>
+</histogram>
+
+<histogram name="Bluetooth.Web.RequestDevice.FilterSize" units="services">
+  <owner>jyasskin@chromium.org</owner>
+  <owner>ortuno@chromium.org</owner>
+  <owner>scheib@chromium.org</owner>
+  <summary>
+    Records how many filters services are used in a filter. The recorded value
+    is a 31-bit hash of the Service UUID. This results will help us better

[Jeffrey Yasskin, 2015/08/11 21:57:50]

This one isn't a hash. ;)

[ortuno, 2015/08/11 22:21:53]

(^_^;)

+    understand the uses of the API and make changes according to developers'
+    behavior.
+  </summary>
+</histogram>
+
+<histogram name="Bluetooth.Web.RequestDevice.OptionalServices.Count"
+    units="filters">
+  <owner>jyasskin@chromium.org</owner>
+  <owner>ortuno@chromium.org</owner>
+  <owner>scheib@chromium.org</owner>
+  <summary>
+    Records how many optional services are used when scanning. This results will
+    help us better understand the uses of the API and make changes according to
+    developers' behavior.
+  </summary>
+</histogram>
+
+<histogram name="Bluetooth.Web.RequestDevice.OptionalServices.Services"
+    units="GATTServiceHash">
+  <owner>jyasskin@chromium.org</owner>
+  <owner>ortuno@chromium.org</owner>
+  <owner>scheib@chromium.org</owner>
+  <summary>
+    Records how many optional services are used when scanning. This results will

[Jeffrey Yasskin, 2015/08/11 21:57:50]

I think you missed mentioning the hash here.

[ortuno, 2015/08/11 22:21:53]

Done.

+    help us better understand the uses of the API and make changes according to
+    developers' behavior.
+  </summary>
+</histogram>
+
+<histogram name="Bluetooth.Web.RequestDevice.Outcome"
+    enum="WebBluetoothRequestDeviceOutcome">
+  <owner>jyasskin@chromium.org</owner>
+  <owner>ortuno@chromium.org</owner>
+  <owner>scheib@chromium.org</owner>
+  <summary>
+    Records the result of a navigator.bluetooth.requestDevice() call. Used to
+    understand what errors developers are getting so we can target efforts
+    toward the most common ones.
+  </summary>
+</histogram>
+
+<histogram name="Bluetooth.Web.RequestDevice.UnionOfServices.Count"
+    units="services">
+  <owner>jyasskin@chromium.org</owner>
+  <owner>ortuno@chromium.org</owner>
+  <owner>scheib@chromium.org</owner>
+  <summary>
+    Records how many services are used when scanning. This results will help us
+    better understand the uses of the API and make changes according to
+    developers' behavior.
+  </summary>
+</histogram>
+
 <histogram name="Bookmarks.LaunchDepth">
   <owner>yfriedman@chromium.org</owner>
   <summary>
@@ -52056,6 +52142,9 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
 </enum>
 
 <enum name="BluetoothRequestDeviceOutcome" type="int">

[ortuno, 2015/08/11 22:21:53]

Removed.

+  <obsolete>
+    Deprecated as of 08/2015.
+  </obsolete>
   <int value="0" label="Success"/>
   <int value="1" label="No Bluetooth adapter"/>
   <int value="2" label="No RenderFrameHost for message source"/>
@@ -59307,6 +59396,41 @@ http://cs/file:chrome/histograms.xml - but prefer this file for new entries.
   <int value="8" label="Overflow"/>
 </enum>
 
+<enum name="GATTServiceHash" type="int">
+  <int value="7464675" label="pulse_oximeter"/>
+  <int value="81094401" label="link_loss"/>
+  <int value="217781951" label="human_interface_device"/>
+  <int value="272218751" label="automation_io"/>
+  <int value="381944619" label="continuous_glucose_monitoring"/>
+  <int value="594122172" label="health_thermometer"/>
+  <int value="690691504" label="bond_management"/>
+  <int value="805645044" label="glucose"/>
+  <int value="836214784" label="cycling_power"/>
+  <int value="869059214" label="next_dst_change"/>
+  <int value="876407493" label="location_and_navigation"/>
+  <int value="890629650" label="running_speed_and_cadence"/>
+  <int value="1029767622" label="internet_protocol_support"/>
+  <int value="1107293675" label="cycling_speed_and_cadence"/>
+  <int value="1188875912" label="phone_alert_status"/>
+  <int value="1219993345" label="device_information"/>
+  <int value="1239202871" label="generic_attribute"/>
+  <int value="1265172773" label="indoor_positioning"/>
+  <int value="1280998251" label="blood_pressure"/>
+  <int value="1369789354" label="current_time"/>
+  <int value="1398099241" label="user_data"/>
+  <int value="1415105941" label="scan_parameters"/>
+  <int value="1424865940" label="alert_notification"/>
+  <int value="1569655226" label="generic_access"/>
+  <int value="1660904097" label="immediate_alert"/>
+  <int value="1663141542" label="battery_service"/>
+  <int value="1696287138" label="weight_scale"/>
+  <int value="1753170093" label="environmental_sensing"/>
+  <int value="1823176820" label="body_composition"/>
+  <int value="1970234661" label="reference_time_update"/>
+  <int value="2075745564" label="heart_rate"/>
+  <int value="2086529225" label="tx_power"/>
+</enum>
+
 <enum name="GCMCheckinRequestStatus" type="int">
   <int value="0" label="Success"/>
   <int value="1" label="URL fetching failed"/>
@@ -71600,6 +71724,17 @@ To add a new entry, add it with any value and run test to compute valid value.
   <int value="5" label="Characteristic.writeValue()"/>
 </enum>
 
+<enum name="WebBluetoothRequestDeviceOutcome" type="int">
+  <int value="0" label="Success"/>
+  <int value="1" label="No Bluetooth adapter"/>
+  <int value="2" label="No RenderFrameHost for message source"/>
+  <int value="3" label="Discovery start failed"/>
+  <int value="4" label="Discovery stop failed"/>
+  <int value="5" label="No matching devices found"/>
+  <int value="6" label="Bluetooth Adapter not present"/>
+  <int value="7" label="Bluetooth Adapter was off"/>
+</enum>
+
 <enum name="WebFontCacheHit" type="int">
   <int value="0" label="Miss"/>
   <int value="1" label="Hit"/>