Implement WebBluetooth getDescriptor[s]

content/browser/bluetooth/bluetooth_metrics.cc

Index: content/browser/bluetooth/bluetooth_metrics.cc
diff --git a/content/browser/bluetooth/bluetooth_metrics.cc b/content/browser/bluetooth/bluetooth_metrics.cc
index dcbc576f88bf9699c31fb8a18c1a3956cd68295f..bf8f34f381695eb04ee8c7a665909a35142049e4 100644
--- a/content/browser/bluetooth/bluetooth_metrics.cc
+++ b/content/browser/bluetooth/bluetooth_metrics.cc
@@ -231,6 +231,7 @@ void RecordGetCharacteristicsOutcome(
                                       UMAGetCharacteristicOutcome::NO_SERVICE);
       return;
     case CacheQueryOutcome::NO_CHARACTERISTIC:
+    case CacheQueryOutcome::NO_DESCRIPTOR:

[ortuno, 2016/12/02 06:14:49]

nit: Similarly here. I don't think we need these changes in this patch.

[dougt, 2016/12/02 18:31:27]

Done.

       NOTREACHED();
       return;
   }
@@ -267,6 +268,10 @@ void RecordGATTOperationOutcome(UMAGATTOperation operation,
     case UMAGATTOperation::START_NOTIFICATIONS:
       RecordStartNotificationsOutcome(outcome);
       return;
+    case UMAGATTOperation::DESCRIPTOR_READ:
+    case UMAGATTOperation::DESCRIPTOR_WRITE:
+      // TODO(crbug.com/667319): Record descriptor interactions.
+      return;
     case UMAGATTOperation::COUNT:
       NOTREACHED();
       return;
@@ -288,6 +293,8 @@ static UMAGATTOperationOutcome TranslateCacheQueryOutcomeToGATTOperationOutcome(
       return UMAGATTOperationOutcome::NO_SERVICE;
     case CacheQueryOutcome::NO_CHARACTERISTIC:
       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;