Bluetooth class changes for implementing local GATT attributes.

device/bluetooth/bluetooth_local_gatt_characteristic.cc

Index: device/bluetooth/bluetooth_local_gatt_characteristic.cc
diff --git a/device/bluetooth/bluetooth_local_gatt_characteristic.cc b/device/bluetooth/bluetooth_local_gatt_characteristic.cc
index 9d4d37aac2372db8fbf23f5ac70740c920cf725a..873307d9eddecd57b0d2ba0072a632ef7010f80d 100644
--- a/device/bluetooth/bluetooth_local_gatt_characteristic.cc
+++ b/device/bluetooth/bluetooth_local_gatt_characteristic.cc
@@ -7,6 +7,10 @@
 #include "base/logging.h"
 #include "device/bluetooth/bluetooth_local_gatt_service.h"
 
+#if defined(OS_CHROMEOS) || defined(OS_LINUX)
+#include "device/bluetooth/bluez/bluetooth_local_gatt_characteristic_bluez.h"
+#endif
+
 namespace device {
 
 BluetoothLocalGattCharacteristic::BluetoothLocalGattCharacteristic() {}
@@ -14,14 +18,18 @@ BluetoothLocalGattCharacteristic::BluetoothLocalGattCharacteristic() {}
 BluetoothLocalGattCharacteristic::~BluetoothLocalGattCharacteristic() {}
 
 // static
-BluetoothLocalGattCharacteristic* BluetoothLocalGattCharacteristic::Create(
-    const BluetoothUUID& uuid,
-    const std::vector<uint8_t>& value,
-    Properties properties,
-    Permissions permissions,
-    BluetoothLocalGattService* service) {
+base::WeakPtr<BluetoothLocalGattCharacteristic>
+BluetoothLocalGattCharacteristic::Create(const BluetoothUUID& uuid,
+                                         Properties properties,
+                                         Permissions permissions,
+                                         BluetoothLocalGattService* service) {
+#if defined(OS_CHROMEOS) || defined(OS_LINUX)
+  return bluez::BluetoothLocalGattCharacteristicBlueZ::Create(

[scheib, 2016/04/26 06:03:32]

Place an impl of BluetoothLocalGattCharacteristic::Create in the
bluetooth_local_gatt_characteristic_bluez.cc file.

#if !defined(OS_CHROMEOS)... this generic impl. 

This is the pattern I see used most other places, incl device/bluetooth. It
avoids the #if defined wrapper around include lines, and once all platforms have
impls then this default impl goes away entirely. 

[rkc, 2016/04/26 18:23:59]

Done.

+      uuid, properties, permissions, service);
+#else
   NOTIMPLEMENTED();
-  return NULL;
+  return nullptr;
+#endif
 }
 
 }  // namespace device