Add BluetoothRemoteGATTDescriptor reporting for .readValue().

content/browser/bluetooth/bluetooth_metrics.cc

 // Copyright 2015 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.
 
 #include "content/browser/bluetooth/bluetooth_metrics.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <map>
@@ skipping 263 matching lines @@
     case UMAGATTOperation::CHARACTERISTIC_READ:
       RecordCharacteristicReadValueOutcome(outcome);
       return;
     case UMAGATTOperation::CHARACTERISTIC_WRITE:
       RecordCharacteristicWriteValueOutcome(outcome);
       return;
     case UMAGATTOperation::START_NOTIFICATIONS:
       RecordStartNotificationsOutcome(outcome);
       return;
     case UMAGATTOperation::DESCRIPTOR_READ:
-      // TODO(667319) Add reporting to descriptors
+      RecordDescriptorReadValueOutcome(outcome);
       return;
     case UMAGATTOperation::COUNT:
       NOTREACHED();
       return;
   }
   NOTREACHED();
 }
 
 static UMAGATTOperationOutcome TranslateCacheQueryOutcomeToGATTOperationOutcome(
     CacheQueryOutcome outcome) {
@@ skipping 11 matching lines @@
       return UMAGATTOperationOutcome::NO_CHARACTERISTIC;
     case CacheQueryOutcome::NO_DESCRIPTOR:
       return UMAGATTOperationOutcome::NO_DESCRIPTOR;
   }
   NOTREACHED() << "No need to record success or renderer crash";
   return UMAGATTOperationOutcome::NOT_SUPPORTED;
 }
 
 // Characteristic.readValue
 
-// static
 void RecordCharacteristicReadValueOutcome(UMAGATTOperationOutcome outcome) {
   UMA_HISTOGRAM_ENUMERATION("Bluetooth.Web.Characteristic.ReadValue.Outcome",
                             static_cast<int>(outcome),
                             static_cast<int>(UMAGATTOperationOutcome::COUNT));
 }
 
 void RecordCharacteristicReadValueOutcome(CacheQueryOutcome outcome) {
   RecordCharacteristicReadValueOutcome(
       TranslateCacheQueryOutcomeToGATTOperationOutcome(outcome));
 }
@@ skipping 22 matching lines @@
 void RecordStartNotificationsOutcome(CacheQueryOutcome outcome) {
   RecordStartNotificationsOutcome(
       TranslateCacheQueryOutcomeToGATTOperationOutcome(outcome));
 }
 
 void RecordRSSISignalStrength(int rssi) {
   UMA_HISTOGRAM_SPARSE_SLOWLY("Bluetooth.Web.RequestDevice.RSSISignalStrength",
                               rssi);
 }
 
+// Descriptor.readValue
+
+void RecordDescriptorReadValueOutcome(UMAGATTOperationOutcome outcome) {
+  UMA_HISTOGRAM_ENUMERATION("Bluetooth.Web.Descriptor.ReadValue.Outcome",
+                            static_cast<int>(outcome),
+                            static_cast<int>(UMAGATTOperationOutcome::COUNT));
+}
+
+void RecordDescriptorReadValueOutcome(CacheQueryOutcome outcome) {
+  RecordDescriptorReadValueOutcome(
+      TranslateCacheQueryOutcomeToGATTOperationOutcome(outcome));
+}
+
 void RecordRSSISignalStrengthLevel(UMARSSISignalStrengthLevel level) {
   UMA_HISTOGRAM_ENUMERATION(
       "Bluetooth.Web.RequestDevice.RSSISignalStrengthLevel",
       static_cast<int>(level),
       static_cast<int>(UMARSSISignalStrengthLevel::COUNT));
 }
 
 void RecordNumOfDevices(bool accept_all_devices, size_t num_of_devices) {
   if (!accept_all_devices) {
     UMA_HISTOGRAM_SPARSE_SLOWLY(
         "Bluetooth.Web.RequestDevice."
         "NumOfDevicesInChooserWhenNotAcceptingAllDevices",
         std::min(num_of_devices, kMaxNumOfDevices));
   }
 }
 
 }  // namespace content