Implement BluetoothRemoteGattCharacteristicWin::GetDescriptors

device/bluetooth/bluetooth_remote_gatt_descriptor_win.cc

+// 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_descriptor_win.h"
+
+#include "base/bind.h"
+#include "device/bluetooth/bluetooth_adapter_win.h"
+#include "device/bluetooth/bluetooth_remote_gatt_characteristic_win.h"
+#include "device/bluetooth/bluetooth_remote_gatt_service_win.h"
+
+namespace device {
+
+BluetoothRemoteGattDescriptorWin::BluetoothRemoteGattDescriptorWin(
+    BluetoothRemoteGattCharacteristicWin* parent_characteristic,
+    BTH_LE_GATT_DESCRIPTOR* descriptor_info,
+    scoped_refptr<base::SequencedTaskRunner>& ui_task_runner)
+    : parent_characteristic_(parent_characteristic),
+      descriptor_info_(descriptor_info),
+      ui_task_runner_(ui_task_runner),
+      weak_ptr_factory_(this) {
+  DCHECK(ui_task_runner_->RunsTasksOnCurrentThread());
+  DCHECK(parent_characteristic_);
+  DCHECK(descriptor_info_.get());
+
+  adapter_ = static_cast<BluetoothAdapterWin*>(
+      parent_characteristic->GetService()->GetDevice()->GetAdapter());
+  DCHECK(adapter_);
+  task_manager_ = adapter_->GetWinBluetoothTaskManager();
+  DCHECK(task_manager_);
+  service_path_ = static_cast<BluetoothRemoteGattServiceWin*>(

[scheib, 2016/03/01 00:06:22]

Static casts are always a red flag and should be minimized. You can create a
GetWinService method, similar to e.g. GetAndroidAdapter().
https://code.google.com/p/chromium/codesearch#chromium/src/device/bluetooth/b...

[gogerald1, 2016/03/01 17:00:13]

Done.

+                      parent_characteristic->GetService())
+                      ->GetServicePath();
+  DCHECK(!service_path_.empty());
+  descriptor_uuid_ =
+      BluetoothTaskManagerWin::BluetoothLowEnergyUuidToBluetoothUuid(
+          descriptor_info_.get()->DescriptorUuid);
+  DCHECK(descriptor_uuid_.IsValid());
+  descriptor_identifier_ = parent_characteristic_->GetIdentifier() + "_" +
+                           std::to_string(descriptor_info_->AttributeHandle);
+}
+
+BluetoothRemoteGattDescriptorWin::~BluetoothRemoteGattDescriptorWin() {
+  DCHECK(ui_task_runner_->RunsTasksOnCurrentThread());
+  adapter_->NotifyGattDescriptorRemoved(this);
+}
+
+std::string BluetoothRemoteGattDescriptorWin::GetIdentifier() const {
+  return descriptor_identifier_;
+}
+
+BluetoothUUID BluetoothRemoteGattDescriptorWin::GetUUID() const {
+  return descriptor_uuid_;
+}
+
+bool BluetoothRemoteGattDescriptorWin::IsLocal() const {
+  return false;
+}
+
+std::vector<uint8_t>& BluetoothRemoteGattDescriptorWin::GetValue() const {
+  NOTIMPLEMENTED();
+  return const_cast<std::vector<uint8_t>&>(descriptor_value_);
+}
+
+BluetoothGattCharacteristic*
+BluetoothRemoteGattDescriptorWin::GetCharacteristic() const {
+  return parent_characteristic_;
+}
+
+BluetoothGattCharacteristic::Permissions
+BluetoothRemoteGattDescriptorWin::GetPermissions() const {
+  NOTIMPLEMENTED();
+  return descriptor_permissions_;
+}
+
+void BluetoothRemoteGattDescriptorWin::ReadRemoteDescriptor(
+    const ValueCallback& callback,
+    const ErrorCallback& error_callback) {
+  DCHECK(ui_task_runner_->RunsTasksOnCurrentThread());
+
+  NOTIMPLEMENTED();
+  error_callback.Run(BluetoothGattService::GATT_ERROR_NOT_SUPPORTED);
+}
+
+void BluetoothRemoteGattDescriptorWin::WriteRemoteDescriptor(
+    const std::vector<uint8_t>& new_value,
+    const base::Closure& callback,
+    const ErrorCallback& error_callback) {
+  DCHECK(ui_task_runner_->RunsTasksOnCurrentThread());
+
+  NOTIMPLEMENTED();
+  error_callback.Run(BluetoothGattService::GATT_ERROR_NOT_SUPPORTED);
+}
+
+uint16_t BluetoothRemoteGattDescriptorWin::GetAttributeHandle() const {
+  return descriptor_info_->AttributeHandle;
+}
+
+}  // namespace device.