bluetooth: Move GetCharacteristic(s) over to Mojo

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 <map>
 #include <set>
@@ skipping 120 matching lines @@
   UMA_HISTOGRAM_ENUMERATION(
       "Bluetooth.Web.GetPrimaryService.Outcome", static_cast<int>(outcome),
       static_cast<int>(UMAGetPrimaryServiceOutcome::COUNT));
 }
 
 void RecordGetPrimaryServiceOutcome(CacheQueryOutcome outcome) {
   DCHECK(outcome == CacheQueryOutcome::NO_DEVICE);
   RecordGetPrimaryServiceOutcome(UMAGetPrimaryServiceOutcome::NO_DEVICE);
 }
 
-// getCharacteristic
+// getCharacteristic & getCharacteristics
 
-void RecordGetCharacteristicOutcome(UMAGetCharacteristicOutcome outcome) {
-  UMA_HISTOGRAM_ENUMERATION(
-      "Bluetooth.Web.GetCharacteristic.Outcome", static_cast<int>(outcome),
-      static_cast<int>(UMAGetCharacteristicOutcome::COUNT));
-}
-
-void RecordGetCharacteristicOutcome(CacheQueryOutcome outcome) {
-  switch (outcome) {
-    case CacheQueryOutcome::SUCCESS:
-    case CacheQueryOutcome::BAD_RENDERER:
-      // No need to record a success or renderer crash.
-      NOTREACHED();
+void RecordGetCharacteristicsOutcome(blink::mojom::WebBluetoothQueryType type,
+                                     UMAGetCharacteristicOutcome outcome) {
+  switch (type) {
+    case blink::mojom::WebBluetoothQueryType::SINGLE:
+      UMA_HISTOGRAM_ENUMERATION(
+          "Bluetooth.Web.GetCharacteristic.Outcome", static_cast<int>(outcome),
+          static_cast<int>(UMAGetCharacteristicOutcome::COUNT));
       return;
-    case CacheQueryOutcome::NO_DEVICE:
-      RecordGetCharacteristicOutcome(UMAGetCharacteristicOutcome::NO_DEVICE);
-      return;
-    case CacheQueryOutcome::NO_SERVICE:
-      RecordGetCharacteristicOutcome(UMAGetCharacteristicOutcome::NO_SERVICE);
-      return;
-    case CacheQueryOutcome::NO_CHARACTERISTIC:
-      NOTREACHED();
+    case blink::mojom::WebBluetoothQueryType::MULTIPLE:
+      UMA_HISTOGRAM_ENUMERATION(
+          "Bluetooth.Web.GetCharacteristics.Outcome", static_cast<int>(outcome),
+          static_cast<int>(UMAGetCharacteristicOutcome::COUNT));
       return;
   }
 }
 
-void RecordGetCharacteristicCharacteristic(const std::string& characteristic) {
-  UMA_HISTOGRAM_SPARSE_SLOWLY("Bluetooth.Web.GetCharacteristic.Characteristic",
-                              HashUUID(characteristic));
-}
-
-// getCharacteristics
-
-void RecordGetCharacteristicsOutcome(UMAGetCharacteristicOutcome outcome) {
-  UMA_HISTOGRAM_ENUMERATION(
-      "Bluetooth.Web.GetCharacteristics.Outcome", static_cast<int>(outcome),
-      static_cast<int>(UMAGetCharacteristicOutcome::COUNT));
-}
-
-void RecordGetCharacteristicsOutcome(CacheQueryOutcome outcome) {
+void RecordGetCharacteristicsOutcome(blink::mojom::WebBluetoothQueryType type,
+                                     CacheQueryOutcome outcome) {
   switch (outcome) {
     case CacheQueryOutcome::SUCCESS:
     case CacheQueryOutcome::BAD_RENDERER:
       // No need to record a success or renderer crash.
       NOTREACHED();
       return;
     case CacheQueryOutcome::NO_DEVICE:
-      RecordGetCharacteristicsOutcome(UMAGetCharacteristicOutcome::NO_DEVICE);
+      RecordGetCharacteristicsOutcome(type,
+                                      UMAGetCharacteristicOutcome::NO_DEVICE);
       return;
     case CacheQueryOutcome::NO_SERVICE:
-      RecordGetCharacteristicsOutcome(UMAGetCharacteristicOutcome::NO_SERVICE);
+      RecordGetCharacteristicsOutcome(type,
+                                      UMAGetCharacteristicOutcome::NO_SERVICE);
       return;
     case CacheQueryOutcome::NO_CHARACTERISTIC:
       NOTREACHED();
       return;
   }
 }
 
-void RecordGetCharacteristicsCharacteristic(const std::string& characteristic) {
-  UMA_HISTOGRAM_SPARSE_SLOWLY("Bluetooth.Web.GetCharacteristics.Characteristic",
-                              HashUUID(characteristic));
+void RecordGetCharacteristicsCharacteristic(
+    blink::mojom::WebBluetoothQueryType type,
+    const std::string& characteristic) {
+  switch (type) {
+    case blink::mojom::WebBluetoothQueryType::SINGLE:
+      UMA_HISTOGRAM_SPARSE_SLOWLY(
+          "Bluetooth.Web.GetCharacteristic.Characteristic",
+          HashUUID(characteristic));
+      return;
+    case blink::mojom::WebBluetoothQueryType::MULTIPLE:
+      UMA_HISTOGRAM_SPARSE_SLOWLY(
+          "Bluetooth.Web.GetCharacteristics.Characteristic",
+          HashUUID(characteristic));
+      return;
+  }
 }
 
 // GATT Operations
 
 void RecordGATTOperationOutcome(UMAGATTOperation operation,
                                 UMAGATTOperationOutcome outcome) {
   switch (operation) {
     case UMAGATTOperation::CHARACTERISTIC_READ:
       RecordCharacteristicReadValueOutcome(outcome);
       return;
@@ skipping 63 matching lines @@
       static_cast<int>(outcome),
       static_cast<int>(UMAGATTOperationOutcome::COUNT));
 }
 
 void RecordStartNotificationsOutcome(CacheQueryOutcome outcome) {
   RecordStartNotificationsOutcome(
       TranslateCacheQueryOutcomeToGATTOperationOutcome(outcome));
 }
 
 }  // namespace content