bluetooth: mac: write characteristic implementation

device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h

 // Copyright 2016 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 DEVICE_BLUETOOTH_BLUETOOTH_REMOTE_GATT_CHARACTERISTIC_MAC_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_REMOTE_GATT_CHARACTERISTIC_MAC_H_
 
 #include "device/bluetooth/bluetooth_remote_gatt_characteristic.h"
 
 #include "base/mac/scoped_nsobject.h"
 
 @class CBCharacteristic;
+typedef NS_ENUM(NSInteger, CBCharacteristicWriteType);

[msarda, 2016/06/27 11:41:36]

Why is this typedef needed? Could CoreBluetooth header be included here to avoid
the redefinition of this enum?

If this is dependent on a build version, then you may use something like:
#if defined(MAC_OS_X_VERSION_10_9) && \
    MAC_OS_X_VERSION_MIN_REQUIRED > MAC_OS_X_VERSION_10_9
#include <CoreBluetooth/CoreBluetooth.h>
#else
@class CBCharacteristic;
typedef NS_ENUM(NSInteger, CBCharacteristicWriteType);
#endif

[jlebel, 2016/06/27 12:20:36]

Done.

 
 namespace device {
 
 class BluetoothRemoteGattServiceMac;
 
 // The BluetoothRemoteGattCharacteristicMac class implements
 // BluetoothRemoteGattCharacteristic for remote GATT services on OS X.
 class DEVICE_BLUETOOTH_EXPORT BluetoothRemoteGattCharacteristicMac
     : public BluetoothRemoteGattCharacteristic {
  public:
@@ skipping 25 matching lines @@
 
   DISALLOW_COPY_AND_ASSIGN(BluetoothRemoteGattCharacteristicMac);
 
  private:
   friend class BluetoothRemoteGattServiceMac;
   friend class BluetoothTestMac;
 
   // Called by the BluetoothRemoteGattServiceMac instance when the
   // characteristics value has been read.
   void DidUpdateValue(NSError* error);
+  // Called by the BluetoothRemoteGattServiceMac instance when the
+  // characteristics value has been written.
+  void DidWriteValue(NSError* error);
   // Returns true if the characteristic is readable.
   bool IsReadable() const;
+  // Returns true if the characteristic is writable.
+  bool IsWritable() const;
+  // Returns the write type (with or without responses).
+  CBCharacteristicWriteType GetCBWriteType() const;
   // Returns CoreBluetooth characteristic.
   CBCharacteristic* GetCBCharacteristic() const;
 
   // gatt_service_ owns instances of this class.
   BluetoothRemoteGattServiceMac* gatt_service_;
   // A characteristic from CBPeripheral.services.characteristics.
   base::scoped_nsobject<CBCharacteristic> cb_characteristic_;
   // Characteristic identifier.
   std::string identifier_;
   // Service UUID.
   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_;
+  // WriteRemoteCharacteristic request callbacks.
+  std::pair<base::Closure, ErrorCallback> write_characteristic_value_callbacks_;
 };
 
 }  // namespace device
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_REMOTE_GATT_CHARACTERISTIC_MAC_H_