chromeos/dbus: Add fake D-Bus clients for GATT client-mode.

chromeos/dbus: Add fake D-Bus clients for GATT client-mode.

Added fake implementations of GATT service, characteristic, and descriptor
clients. A new "Low Energy" fake device is introduced, which uses the fake
GATT clients to simulate the Heart Rate Service.

BUG=351229
TEST=1. device_unittests, chromeos_unittests.
2. Build and run chrome for Linux with "chromeos=1". Run Bluetooth device
discovery, connect to the device named "Bluetooth 4.0 Heart Rate Monitor".
Watch the debug output (vmodule="*bluetooth_gatt*=2") for the GATT clients,
which should create fake heart rate service and descriptors and update the
measurement value at regular intervals.

R=keybuk@chromium.org, stevenjb@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=259022

[armansito, 2014-03-21 01:30:12]

stevenjb@: rubberstamp for chromeos/dbus/dbus_thread_manager.* and
chromeos/dbus/fake_dbus_thread_manager.*

keybuk@: All else.

[stevenjb, 2014-03-21 17:26:05]

owner lgtm

https://codereview.chromium.org/206443009/diff/1/chromeos/chromeos.gyp
File chromeos/chromeos.gyp (right):

https://codereview.chromium.org/206443009/diff/1/chromeos/chromeos.gyp#newcod...
chromeos/chromeos.gyp:124: 'dbus/fake_bluetooth_gatt_service_client.h',
At this point we really need a dbus/bluetooth directory. Probably one for shill,
power, and nfc too. We should do that in a follow up CL.

[keybuk, 2014-03-21 19:57:53]

https://codereview.chromium.org/206443009/diff/1/chromeos/dbus/fake_bluetooth...
File chromeos/dbus/fake_bluetooth_device_client.cc (right):

https://codereview.chromium.org/206443009/diff/1/chromeos/dbus/fake_bluetooth...
chromeos/dbus/fake_bluetooth_device_client.cc:686:
dbus::ObjectPath(kLowEnergyPath));
Since this is an LE device, following the pattern of LE discovery would be
better ...

It should be created on the first step, and be "changed" every subsequent step

That way it also satisfies other possible test cases for Low Energy devices

[armansito, 2014-03-21 23:18:10]

https://codereview.chromium.org/206443009/diff/1/chromeos/dbus/fake_bluetooth...
File chromeos/dbus/fake_bluetooth_device_client.cc (right):

https://codereview.chromium.org/206443009/diff/1/chromeos/dbus/fake_bluetooth...
chromeos/dbus/fake_bluetooth_device_client.cc:686:
dbus::ObjectPath(kLowEnergyPath));
On 2014/03/21 19:57:53, keybuk wrote:
> Since this is an LE device, following the pattern of LE discovery would be
> better ...
> 
> It should be created on the first step, and be "changed" every subsequent step
> 
> That way it also satisfies other possible test cases for Low Energy devices

Done.

[keybuk, 2014-03-21 23:57:33]

lgtm

https://codereview.chromium.org/206443009/diff/20001/chromeos/dbus/fake_bluet...
File chromeos/dbus/fake_bluetooth_device_client.cc (right):

https://codereview.chromium.org/206443009/diff/20001/chromeos/dbus/fake_bluet...
chromeos/dbus/fake_bluetooth_device_client.cc:680:
UpdateDeviceRSSI(dbus::ObjectPath(kLowEnergyPath), base::RandInt(-90, -30));
nit: use constants

[armansito, 2014-03-22 02:21:59]

https://codereview.chromium.org/206443009/diff/20001/chromeos/dbus/fake_bluet...
File chromeos/dbus/fake_bluetooth_device_client.cc (right):

https://codereview.chromium.org/206443009/diff/20001/chromeos/dbus/fake_bluet...
chromeos/dbus/fake_bluetooth_device_client.cc:680:
UpdateDeviceRSSI(dbus::ObjectPath(kLowEnergyPath), base::RandInt(-90, -30));
On 2014/03/21 23:57:33, keybuk wrote:
> nit: use constants

Done.

[armansito, 2014-03-24 21:51:37]

Committed patchset #5 manually as r259022 (presubmit successful).