Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3281)

Unified Diff: device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h

Issue 2052513002: Read characteristic implementation on macOS (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@characteristicscan_servicescan_cleanup
Patch Set: Addressing msarda's comment Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h
diff --git a/device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h b/device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h
index 57d0aad0b74492e02831ff2231fc8368d49b88b2..d591f174aa4b6b10477995e66a71020a6283bd92 100644
--- a/device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h
+++ b/device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h
@@ -52,6 +52,11 @@ class DEVICE_BLUETOOTH_EXPORT BluetoothRemoteGattCharacteristicMac
friend class BluetoothRemoteGattServiceMac;
friend class BluetoothTestMac;
+ // Called by the BluetoothRemoteGattServiceMac instance when the
+ // characteristics value has been read.
+ void DidUpdateValue(NSError* error);
+ // Returns true if the characteristic is readable.
+ bool IsReadable() const;
// Returns CoreBluetooth characteristic.
CBCharacteristic* GetCBCharacteristic() const;
@@ -65,6 +70,10 @@ class DEVICE_BLUETOOTH_EXPORT BluetoothRemoteGattCharacteristicMac
BluetoothUUID uuid_;
// Characteristic value.
std::vector<uint8_t> value_;
+ // True if a gatt read or write request is in progress.
+ bool characteristic_value_read_or_write_in_progress_;
+ // ReadRemoteCharacteristic request callbacks.
+ std::pair<ValueCallback, ErrorCallback> read_characteristic_value_callbacks_;
};
} // namespace device

Powered by Google App Engine
This is Rietveld 408576698