Bluetooth class changes for implementing local GATT attributes.

device/bluetooth/bluez/bluetooth_gatt_characteristic_bluez.h

Index: device/bluetooth/bluez/bluetooth_gatt_characteristic_bluez.h
diff --git a/device/bluetooth/bluez/bluetooth_gatt_characteristic_bluez.h b/device/bluetooth/bluez/bluetooth_gatt_characteristic_bluez.h
index 4c0848c05e8502b459d877bdec1f69efcefa21e1..0f0ae2991815497c9adf5f5689b9b4f8d80c68b3 100644
--- a/device/bluetooth/bluez/bluetooth_gatt_characteristic_bluez.h
+++ b/device/bluetooth/bluez/bluetooth_gatt_characteristic_bluez.h
@@ -27,14 +27,13 @@ class BluetoothGattCharacteristicBlueZ
   const dbus::ObjectPath& object_path() const { return object_path_; }
 
  protected:
-  explicit BluetoothGattCharacteristicBlueZ(
-      const dbus::ObjectPath& object_path);
+  BluetoothGattCharacteristicBlueZ();

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

I mentioned in the earlier patch (before splitting to this) that it seems odd to
move the members that are assigned during construction up to be protected. At
the time I suggested having virtual implementations of e.g. object_path.

You argued against virtual impl of the accessors, which is OK. But, I still
don't like the uncertain constructed state of these members.

object_path_ can remain a constructor argument and the member can be initialized
programmatically in the constructor initializer list. E.g. below 
  object_path_(BluetoothLocalGattServiceBlueZ::AddGuidToObjectPath(
    service->object_path().value() + "/characteristic"))

This makes it clear in the BluetoothGattCharacteristicBlueZ declaration that
object_path_ will always be set upon construction by any subclass.

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

Done.

   ~BluetoothGattCharacteristicBlueZ() override;
 
- private:
   // Object path of the D-Bus characteristic object.
   dbus::ObjectPath object_path_;
 
+ private:
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<BluetoothGattCharacteristicBlueZ> weak_ptr_factory_;