Add BluetoothRemoteGATTDescriptor reporting for .writeValue().

content/browser/bluetooth/bluetooth_metrics.h

 // 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.
 
 #ifndef CONTENT_BROWSER_BLUETOOTH_BLUETOOTH_METRICS_H_
 #define CONTENT_BROWSER_BLUETOOTH_BLUETOOTH_METRICS_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 269 matching lines @@
 // Records the outcome of a cache query for startNotifications. Should only be
 // called if QueryCacheForCharacteristic fails.
 void RecordStartNotificationsOutcome(CacheQueryOutcome outcome);
 
 // Descriptor.readValue() Metrics
 // There should be a call to this function for every call to
 // Send(BluetoothMsg_ReadDescriptorValueSuccess) and
 // Send(BluetoothMsg_ReadDescriptorValueError).
 void RecordDescriptorReadValueOutcome(UMAGATTOperationOutcome error);
 
+// Records the outcome of a cache query for writeValue. Should only be called if

[ortuno, 2017/01/31 23:50:37]

You changed the wrong comment hehe. This should be readValue. The one in the new
code should be writeValue.

[dougt, 2017/01/31 23:54:35]

yup. :/

+// QueryCacheForDescriptor fails.
+void RecordDescriptorReadValueOutcome(CacheQueryOutcome outcome);
+
+// Descriptor.writeValue() Metrics
+// There should be a call to this function for every call to
+// Send(BluetoothMsg_ReadDescriptorValueSuccess) and
+// Send(BluetoothMsg_ReadDescriptorValueError).
+void RecordDescriptorWriteValueOutcome(UMAGATTOperationOutcome error);
+
 // Records the outcome of a cache query for readValue. Should only be called if
 // QueryCacheForDescriptor fails.
-void RecordDescriptorReadValueOutcome(CacheQueryOutcome outcome);
+void RecordDescriptorWriteValueOutcome(CacheQueryOutcome outcome);
 
 enum class UMARSSISignalStrengthLevel {
   LESS_THAN_OR_EQUAL_TO_MIN_RSSI,
   LEVEL_0,
   LEVEL_1,
   LEVEL_2,
   LEVEL_3,
   LEVEL_4,
   GREATER_THAN_OR_EQUAL_TO_MAX_RSSI,
   // Note: Add new RSSI signal strength level immediately above this line.
   COUNT
 };
 
 // Records the raw RSSI, and processed result displayed to users, when
 // content::BluetoothDeviceChooserController::CalculateSignalStrengthLevel() is
 // called.
 void RecordRSSISignalStrength(int rssi);
 void RecordRSSISignalStrengthLevel(UMARSSISignalStrengthLevel level);
 
 // In the case of not accepting all devices in the options that are given
 // to WebBluetooth requestDevice(), records the number of devices in the
 // chooser when a device is paired.
 void RecordNumOfDevices(bool accept_all_devices, size_t num_of_devices);
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_BLUETOOTH_BLUETOOTH_METRICS_H_