Add BluetoothRemoteGATTDescriptor reporting for .writeValue().

content/browser/bluetooth/bluetooth_metrics.h

Index: content/browser/bluetooth/bluetooth_metrics.h
diff --git a/content/browser/bluetooth/bluetooth_metrics.h b/content/browser/bluetooth/bluetooth_metrics.h
index 55aac46cf83719c64b30920274b1e4721fc4baa1..37bbbc8d60db4470f5788ba83f3c30771a74a1aa 100644
--- a/content/browser/bluetooth/bluetooth_metrics.h
+++ b/content/browser/bluetooth/bluetooth_metrics.h
@@ -287,10 +287,20 @@ void RecordStartNotificationsOutcome(CacheQueryOutcome outcome);
 // Send(BluetoothMsg_ReadDescriptorValueError).
 void RecordDescriptorReadValueOutcome(UMAGATTOperationOutcome error);
 
-// Records the outcome of a cache query for readValue. Should only be called if
+// 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 RecordDescriptorWriteValueOutcome(CacheQueryOutcome outcome);
+
 enum class UMARSSISignalStrengthLevel {
   LESS_THAN_OR_EQUAL_TO_MIN_RSSI,
   LEVEL_0,