bluetooth: macOS: Adding histograms for NSError values

device/bluetooth/bluetooth_low_energy_device_mac.mm

 // 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 "device/bluetooth/bluetooth_low_energy_device_mac.h"
 
 #import <CoreFoundation/CoreFoundation.h>
 #include <stddef.h>
 
 #include "base/mac/mac_util.h"
 #include "base/mac/scoped_cftyperef.h"
 #include "base/mac/sdk_forward_declarations.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/sys_string_conversions.h"
 #include "device/bluetooth/bluetooth_adapter_mac.h"
+#include "device/bluetooth/bluetooth_adapter_mac_metrics.h"
 #include "device/bluetooth/bluetooth_device.h"
 #include "device/bluetooth/bluetooth_low_energy_peripheral_delegate.h"
 #include "device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h"
 #include "device/bluetooth/bluetooth_remote_gatt_descriptor_mac.h"
 #include "device/bluetooth/bluetooth_remote_gatt_service_mac.h"
 
 using device::BluetoothDevice;
 using device::BluetoothLowEnergyDeviceMac;
 
 BluetoothLowEnergyDeviceMac::BluetoothLowEnergyDeviceMac(
@@ skipping 181 matching lines @@
     VLOG(1) << *this
             << ": BluetoothLowEnergyDeviceMac::discovery_pending_count_ "
             << discovery_pending_count_;
     discovery_pending_count_ = 0;
     return;
   }
   if (error) {
     // TODO(http://crbug.com/609320): Need to pass the error.
     // TODO(http://crbug.com/609844): Decide what to do if discover failed
     // a device services.
+    LogNSErrorToHistogram(
+        error, WebBluetoothMacOSAPIs::
+                   WEB_BLUETOOTH_MACOS_APIS_DID_DISCOVER_PRIMARY_SERVICES);
     VLOG(1) << *this << ": Can't discover primary services: "
             << BluetoothAdapterMac::String(error);
     return;
   }
 
   if (!IsGattConnected()) {
     // Don't create services if the device disconnected.
     VLOG(1) << *this << ": DidDiscoverPrimaryServices, gatt not connected.";
     return;
   }
@@ skipping 23 matching lines @@
       gatt_service_mac->DiscoverCharacteristics();
     }
     SendNotificationIfDiscoveryComplete();
   }
 }
 
 void BluetoothLowEnergyDeviceMac::DidDiscoverCharacteristics(
     CBService* cb_service,
     NSError* error) {
   if (error) {
+    LogNSErrorToHistogram(
+        error, WebBluetoothMacOSAPIs::
+                   WEB_BLUETOOTH_MACOS_APIS_DID_DISCOVER_CHARACTERISTICS);
     // TODO(http://crbug.com/609320): Need to pass the error.
     // TODO(http://crbug.com/609844): Decide what to do if discover failed
     VLOG(1) << *this << ": Can't discover characteristics: "
             << BluetoothAdapterMac::String(error);
     return;
   }
 
   if (!IsGattConnected()) {
     VLOG(1) << *this << ": DidDiscoverCharacteristics, gatt disconnected.";
     // Don't create characteristics if the device disconnected.
@@ skipping 59 matching lines @@
   BluetoothRemoteGattServiceMac* gatt_service =
       GetBluetoothRemoteGattService(characteristic.service);
   DCHECK(gatt_service);
   gatt_service->DidUpdateNotificationState(characteristic, error);
 }
 
 void BluetoothLowEnergyDeviceMac::DidDiscoverDescriptors(
     CBCharacteristic* cb_characteristic,
     NSError* error) {
   if (error) {
+    LogNSErrorToHistogram(
+        error, WebBluetoothMacOSAPIs::
+                   WEB_BLUETOOTH_MACOS_APIS_DID_DISCOVER_DESCRIPTORS);
     // TODO(http://crbug.com/609320): Need to pass the error.
     // TODO(http://crbug.com/609844): Decide what to do if discover failed
     VLOG(1) << *this << ": Can't discover descriptors: "
             << BluetoothAdapterMac::String(error);
     return;
   }
   if (!IsGattConnected()) {
     VLOG(1) << *this << ": DidDiscoverDescriptors, disconnected.";
     // Don't discover descriptors if the device disconnected.
     return;
@@ skipping 126 matching lines @@
   return gatt_characteristic->GetBluetoothRemoteGattDescriptorMac(
       cb_descriptor);
 }
 
 void BluetoothLowEnergyDeviceMac::DidDisconnectPeripheral(NSError* error) {
   connected_ = false;
   VLOG(1) << *this << ": Disconnected from peripheral.";
   if (error) {
     VLOG(1) << *this
             << ": Bluetooth error: " << BluetoothAdapterMac::String(error);
+    LogNSErrorToHistogram(
+        error, WebBluetoothMacOSAPIs::
+                   WEB_BLUETOOTH_MACOS_APIS_DID_DISCONNECT_PERIPHERAL);
   }
   SetGattServicesDiscoveryComplete(false);
   // Removing all services at once to ensure that calling GetGattService on
   // removed service in GattServiceRemoved returns null.
   GattServiceMap gatt_services_swapped;
   gatt_services_swapped.swap(gatt_services_);
   gatt_services_swapped.clear();
   device_uuids_.ClearServiceUUIDs();
   // There are two cases in which this function will be called:
   //   1. When the connection to the device breaks (either because
@@ skipping 16 matching lines @@
   // TODO(crbug.com/703878): Should use
   // BluetoothLowEnergyDeviceMac::GetNameForDisplay() instead.
   base::Optional<std::string> name = device.GetName();
   const char* is_gatt_connected =
       device.IsGattConnected() ? "GATT connected" : "GATT disconnected";
   return out << "<BluetoothLowEnergyDeviceMac " << device.GetAddress() << "/"
              << &device << ", " << is_gatt_connected << ", \""
              << name.value_or("Unnamed device") << "\">";
 }
 }  // namespace device