bluetooth: mac: write characteristic implementation

device/bluetooth/bluetooth_remote_gatt_characteristic_mac.mm

 // 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.
 
 #include "device/bluetooth/bluetooth_remote_gatt_characteristic_mac.h"
 
 #import <CoreBluetooth/CoreBluetooth.h>
 
 #include "base/bind.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 144 matching lines @@
   read_characteristic_value_callbacks_ =
       std::make_pair(callback, error_callback);
   [gatt_service_->GetCBPeripheral()
       readValueForCharacteristic:cb_characteristic_];
 }
 
 void BluetoothRemoteGattCharacteristicMac::WriteRemoteCharacteristic(
     const std::vector<uint8_t>& new_value,
     const base::Closure& callback,
     const ErrorCallback& error_callback) {
-  NOTIMPLEMENTED();
+  if (!IsWritable()) {
+    base::ThreadTaskRunnerHandle::Get()->PostTask(
+        FROM_HERE,
+        base::Bind(error_callback,
+                   BluetoothRemoteGattService::GATT_ERROR_NOT_PERMITTED));
+    return;
+  }
+  if (characteristic_value_read_or_write_in_progress_) {
+    base::ThreadTaskRunnerHandle::Get()->PostTask(
+        FROM_HERE,
+        base::Bind(error_callback,
+                   BluetoothRemoteGattService::GATT_ERROR_IN_PROGRESS));
+    return;
+  }
+  characteristic_value_read_or_write_in_progress_ = true;
+  write_characteristic_value_callbacks_ =
+      std::make_pair(callback, error_callback);
+  base::scoped_nsobject<NSData> nsdata_value(
+      [[NSData alloc] initWithBytes:new_value.data() length:new_value.size()]);
+  CBCharacteristicWriteType write_type = GetCBWriteType();
+  [gatt_service_->GetCBPeripheral() writeValue:nsdata_value
+                             forCharacteristic:cb_characteristic_
+                                          type:write_type];
+  if (write_type == CBCharacteristicWriteWithoutResponse) {
+    DidWriteValue(nil);

[ortuno, 2016/06/22 21:37:45]

We still want this function to be async so you need to use PostTask

[jlebel, 2016/06/23 18:33:01]

Done.

+  }
 }
 
 void BluetoothRemoteGattCharacteristicMac::DidUpdateValue(NSError* error) {
   if (!characteristic_value_read_or_write_in_progress_) {
     return;
   }
   std::pair<ValueCallback, ErrorCallback> callbacks;
   callbacks.swap(read_characteristic_value_callbacks_);
   characteristic_value_read_or_write_in_progress_ = false;
   if (error) {
     VLOG(1) << "Bluetooth error while reading for characteristic, domain: "
             << error.domain.UTF8String << ", error code: " << error.code;
     BluetoothGattService::GattErrorCode error_code =
         BluetoothDeviceMac::GetGattErrorCodeFromNSError(error);
     callbacks.second.Run(error_code);
     return;
   }
   NSData* nsdata_value = cb_characteristic_.get().value;
   const uint8_t* buffer = static_cast<const uint8_t*>(nsdata_value.bytes);
   value_.assign(buffer, buffer + nsdata_value.length);
   gatt_service_->GetMacAdapter()->NotifyGattCharacteristicValueChanged(this,
                                                                        value_);
   callbacks.first.Run(value_);
 }
 
+void BluetoothRemoteGattCharacteristicMac::DidWriteValue(NSError* error) {
+  if (!characteristic_value_read_or_write_in_progress_) {

[ortuno, 2016/06/22 21:37:45]

Please add a comment explaining when would this function get called if neither
read nor write are in progress.

[jlebel, 2016/06/23 18:33:01]

I'm not sure when it would be the case. For me it close to be a DCHECK(), unless
a buggy device can send 2 confirmations of the same write (same for the read).

[ortuno, 2016/06/23 19:21:16]

Now that I think about it we have the same line in Android in case of a buggy
device sending two confirmations. Could you add the bit about the buggy device
to the comment?

[jlebel, 2016/06/23 19:28:29]

I'm not an expert in the BLE protocol, I don't known if it is possible. But I
did copy this pattern from Android or windows. I don't know if the macOS would
catch it or not.

+    return;
+  }
+  std::pair<base::Closure, ErrorCallback> callbacks;
+  callbacks.swap(write_characteristic_value_callbacks_);
+  characteristic_value_read_or_write_in_progress_ = false;
+  if (error) {
+    VLOG(1) << "Bluetooth error while reading for characteristic, domain: "
+            << error.domain.UTF8String << ", error code: " << error.code;
+    BluetoothGattService::GattErrorCode error_code =
+        BluetoothDeviceMac::GetGattErrorCodeFromNSError(error);
+    callbacks.second.Run(error_code);
+    return;
+  }
+  NSData* nsdata_value = cb_characteristic_.get().value;
+  const uint8_t* buffer = static_cast<const uint8_t*>(nsdata_value.bytes);
+  std::vector<uint8_t> gatt_value(buffer, buffer + nsdata_value.length);
+  gatt_service_->GetMacAdapter()->NotifyGattCharacteristicValueChanged(this,
+                                                                       value_);
+  callbacks.first.Run();
+}
+
 bool BluetoothRemoteGattCharacteristicMac::IsReadable() const {
   return GetProperties() & BluetoothGattCharacteristic::PROPERTY_READ;
 }
 
+bool BluetoothRemoteGattCharacteristicMac::IsWritable() const {
+  BluetoothGattCharacteristic::Properties properties = GetProperties();
+  return (properties & BluetoothGattCharacteristic::PROPERTY_WRITE) ||
+         (properties & PROPERTY_WRITE_WITHOUT_RESPONSE);
+}
+
+CBCharacteristicWriteType BluetoothRemoteGattCharacteristicMac::GetCBWriteType()
+    const {
+  return (GetProperties() & BluetoothGattCharacteristic::PROPERTY_WRITE)
+             ? CBCharacteristicWriteWithResponse
+             : CBCharacteristicWriteWithoutResponse;
+}
+
 CBCharacteristic* BluetoothRemoteGattCharacteristicMac::GetCBCharacteristic()
     const {
   return cb_characteristic_.get();
 }
-
 }  // namespace device.