bluetooth: macOS: Adding histograms for NSError values

device/bluetooth/bluetooth_remote_gatt_descriptor_mac.mm

 // Copyright 2017 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_remote_gatt_descriptor_mac.h"
 
 #include "base/bind.h"
 #import "base/mac/foundation_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/threading/thread_task_runner_handle.h"
 #import "device/bluetooth/bluetooth_adapter_mac.h"
+#include "device/bluetooth/bluetooth_adapter_mac_metrics.h"
 #import "device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h"
 
 using base::mac::ObjCCast;
 
 namespace device {
 
 std::vector<uint8_t> VectorValueFromObjC(id objc_value) {
   // According to
   // https://developer.apple.com/reference/corebluetooth/cbdescriptor some
   // descriptor values can be NSData, NSString or NSNumber.
@@ skipping 111 matching lines @@
 void BluetoothRemoteGattDescriptorMac::DidUpdateValueForDescriptor(
     NSError* error) {
   if (!value_read_or_write_in_progress_) {
     VLOG(1) << *this << ": Value updated, no read in progress.";
     return;
   }
   std::pair<ValueCallback, ErrorCallback> callbacks;
   callbacks.swap(read_value_callbacks_);
   value_read_or_write_in_progress_ = false;
   if (error) {
+    LogNSErrorToHistogram(
+        error, WebBluetoothMacOSAPIs::
+                   WEB_BLUETOOTH_MACOS_APIS_DID_UPDATE_VALUE_FOR_DESCRIPTOR);
     BluetoothGattService::GattErrorCode error_code =
         BluetoothDeviceMac::GetGattErrorCodeFromNSError(error);
     VLOG(1) << *this << ": Read value failed with error: "
             << BluetoothAdapterMac::String(error)
             << ", converted to error code: " << error_code;
     callbacks.second.Run(error_code);
     return;
   }
   VLOG(1) << *this << ": Value read.";
   value_ = VectorValueFromObjC([cb_descriptor_ value]);
   callbacks.first.Run(value_);
 }
 
 void BluetoothRemoteGattDescriptorMac::DidWriteValueForDescriptor(
     NSError* error) {
   if (!value_read_or_write_in_progress_) {
     VLOG(1) << *this << ": Value written, no write in progress.";
     return;
   }
   std::pair<base::Closure, ErrorCallback> callbacks;
   callbacks.swap(write_value_callbacks_);
   value_read_or_write_in_progress_ = false;
   if (error) {
+    LogNSErrorToHistogram(
+        error, WebBluetoothMacOSAPIs::
+                   WEB_BLUETOOTH_MACOS_APIS_DID_WRITE_VALUE_FOR_DESCRIPTOR);
     BluetoothGattService::GattErrorCode error_code =
         BluetoothDeviceMac::GetGattErrorCodeFromNSError(error);
     VLOG(1) << *this << ": Write value failed with error: "
             << BluetoothAdapterMac::String(error)
             << ", converted to error code: " << error_code;
     callbacks.second.Run(error_code);
     return;
   }
   VLOG(1) << *this << ": Value written.";
   callbacks.first.Run();
@@ skipping 14 matching lines @@
       static_cast<const BluetoothRemoteGattCharacteristicMac*>(
           descriptor.GetCharacteristic());
   return out << "<BluetoothRemoteGattDescriptorMac "
              << descriptor.GetUUID().canonical_value() << "/" << &descriptor
              << ", characteristic: "
              << characteristic_mac->GetUUID().canonical_value() << "/"
              << characteristic_mac << ">";
 }
 
 }  // namespace device.