Bluetooth class changes for implementing local GATT attributes.

device/bluetooth/bluez/bluetooth_local_gatt_characteristic_bluez.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/bluez/bluetooth_local_gatt_characteristic_bluez.h"
 
 #include <string>
 
 #include "base/logging.h"
+#include "base/memory/ptr_util.h"
 #include "device/bluetooth/bluez/bluetooth_local_gatt_service_bluez.h"
 
+namespace device {
+
+// static
+base::WeakPtr<BluetoothLocalGattCharacteristic>
+BluetoothLocalGattCharacteristic::Create(const BluetoothUUID& uuid,
+                                         Properties properties,
+                                         Permissions permissions,
+                                         BluetoothLocalGattService* service) {
+  return bluez::BluetoothLocalGattCharacteristicBlueZ::Create(
+      uuid, properties, permissions, service);
+}
+
+}  // device
+
 namespace bluez {
 
+// static
+base::WeakPtr<device::BluetoothLocalGattCharacteristic>
+BluetoothLocalGattCharacteristicBlueZ::Create(

[scheib, 2016/04/28 01:06:24]

Two ::Create methods are not needed, the ___BlueZ::Create can be removed and
code moved to the ___::Create method. (ditto descriptor)

[rkc, 2016/04/28 02:06:58]

Done.

+    const device::BluetoothUUID& uuid,
+    device::BluetoothGattCharacteristic::Properties properties,
+    device::BluetoothGattCharacteristic::Permissions permissions,
+    device::BluetoothLocalGattService* service) {
+  DCHECK(service);
+  BluetoothLocalGattServiceBlueZ* service_bluez =
+      static_cast<BluetoothLocalGattServiceBlueZ*>(service);
+  BluetoothLocalGattCharacteristicBlueZ* characteristic =
+      new BluetoothLocalGattCharacteristicBlueZ(uuid, service_bluez);
+  service_bluez->AddCharacteristic(base::WrapUnique(characteristic));
+  return characteristic->weak_ptr_factory_.GetWeakPtr();
+}
+
 BluetoothLocalGattCharacteristicBlueZ::BluetoothLocalGattCharacteristicBlueZ(
-    BluetoothLocalGattServiceBlueZ* service,
-    const dbus::ObjectPath& object_path)
-    : BluetoothGattCharacteristicBlueZ(object_path), weak_ptr_factory_(this) {
+    const device::BluetoothUUID& uuid,
+    BluetoothLocalGattServiceBlueZ* service)
+    : uuid_(uuid), service_(service), weak_ptr_factory_(this) {
+  object_path_ = BluetoothLocalGattServiceBlueZ::AddGuidToObjectPath(
+      service->object_path().value() + "/characteristic");
   VLOG(1) << "Creating local GATT characteristic with identifier: "
           << GetIdentifier();
 }
 
 BluetoothLocalGattCharacteristicBlueZ::
     ~BluetoothLocalGattCharacteristicBlueZ() {}
 
+device::BluetoothUUID BluetoothLocalGattCharacteristicBlueZ::GetUUID() const {
+  return uuid_;
+}
+
+device::BluetoothGattCharacteristic::Properties
+BluetoothLocalGattCharacteristicBlueZ::GetProperties() const {
+  NOTIMPLEMENTED();
+  return Properties();
+}
+
+device::BluetoothGattCharacteristic::Permissions
+BluetoothLocalGattCharacteristicBlueZ::GetPermissions() const {
+  NOTIMPLEMENTED();
+  return Permissions();
+}
+
+BluetoothLocalGattServiceBlueZ*
+BluetoothLocalGattCharacteristicBlueZ::GetService() {
+  return service_;
+}
+
+void BluetoothLocalGattCharacteristicBlueZ::AddDescriptor(
+    std::unique_ptr<BluetoothLocalGattDescriptorBlueZ> descriptor) {
+  descriptors_.push_back(std::move(descriptor));
+}
+
+const std::vector<std::unique_ptr<BluetoothLocalGattDescriptorBlueZ>>&
+BluetoothLocalGattCharacteristicBlueZ::GetDescriptors() const {
+  return descriptors_;
+}
+
 }  // namespace bluez