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 RecordGetCharacteristicsOutcome(bool single_characteristic,
+                                     UMAGetCharacteristicOutcome outcome) {
+  if (single_characteristic) {
+    UMA_HISTOGRAM_ENUMERATION(
+        "Bluetooth.Web.GetCharacteristic.Outcome", static_cast<int>(outcome),
+        static_cast<int>(UMAGetCharacteristicOutcome::COUNT));
+  } else {
+    UMA_HISTOGRAM_ENUMERATION(
+        "Bluetooth.Web.GetCharacteristics.Outcome", static_cast<int>(outcome),
+        static_cast<int>(UMAGetCharacteristicOutcome::COUNT));
+  }
 }
 
-void RecordGetCharacteristicOutcome(CacheQueryOutcome outcome) {
+void RecordGetCharacteristicsOutcome(bool single_characteristic,
+                                     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:
-      RecordGetCharacteristicOutcome(UMAGetCharacteristicOutcome::NO_DEVICE);
-      return;
-    case CacheQueryOutcome::NO_SERVICE:
-      RecordGetCharacteristicOutcome(UMAGetCharacteristicOutcome::NO_SERVICE);
-      return;
-    case CacheQueryOutcome::NO_CHARACTERISTIC:
-      NOTREACHED();
-      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) {
-  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(single_characteristic,
+                                      UMAGetCharacteristicOutcome::NO_DEVICE);
       return;
     case CacheQueryOutcome::NO_SERVICE:
-      RecordGetCharacteristicsOutcome(UMAGetCharacteristicOutcome::NO_SERVICE);
+      RecordGetCharacteristicsOutcome(single_characteristic,
+                                      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(bool single_characteristic,
+                                            const std::string& characteristic) {
+  if (single_characteristic) {
+    UMA_HISTOGRAM_SPARSE_SLOWLY(
+        "Bluetooth.Web.GetCharacteristic.Characteristic",
+        HashUUID(characteristic));
+  } else {
+    UMA_HISTOGRAM_SPARSE_SLOWLY(
+        "Bluetooth.Web.GetCharacteristics.Characteristic",
+        HashUUID(characteristic));
+  }
 }
 
 // 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