chrome.bluetoothLowEnergy: Implement getCharacteristics.

chrome.bluetoothLowEnergy: Implement getCharacteristics.

This CL implements the getCharacteristics method of the bluetoothLowEnergy API
and adds a mapping for Characteristic instance IDs to
BluetoothLowEnergyEventRouter.

BUG=265663
TEST=browser_tests --gtest_filter=BluetoothLowEnergyApiTest.*

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

[rpaquay, 2014-04-29 17:38:30]

lgtm

https://codereview.chromium.org/255053002/diff/1/chrome/browser/extensions/ap...
File
chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc
(right):

https://codereview.chromium.org/255053002/diff/1/chrome/browser/extensions/ap...
chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc:44:
// because Characteristic contains a list of enum CharacteristicProperty.
I would open a bug for this, it sounds like an oversight.

[armansito, 2014-04-29 19:31:18]

https://codereview.chromium.org/255053002/diff/1/chrome/browser/extensions/ap...
File
chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc
(right):

https://codereview.chromium.org/255053002/diff/1/chrome/browser/extensions/ap...
chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc:44:
// because Characteristic contains a list of enum CharacteristicProperty.
On 2014/04/29 17:38:30, rpaquay wrote:
> I would open a bug for this, it sounds like an oversight.

Filed bug and added a TODO here.

[rpaquay, 2014-04-29 19:33:11]

https://codereview.chromium.org/255053002/diff/1/chrome/browser/extensions/ap...
File
chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc
(right):

https://codereview.chromium.org/255053002/diff/1/chrome/browser/extensions/ap...
chrome/browser/extensions/api/bluetooth_low_energy/bluetooth_low_energy_api.cc:44:
// because Characteristic contains a list of enum CharacteristicProperty.
On 2014/04/29 19:31:19, armansito wrote:
> On 2014/04/29 17:38:30, rpaquay wrote:
> > I would open a bug for this, it sounds like an oversight.
> 
> Filed bug and added a TODO here.

Thanks!

[keybuk, 2014-04-30 16:29:47]

lgtm

[armansito, 2014-05-02 22:24:49]

The CQ bit was checked by armansito@chromium.org

[commit-bot: I haz the power, 2014-05-02 22:25:13]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/armansito@chromium.org/255053002/40001

[commit-bot: I haz the power, 2014-05-02 23:34:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-02 23:34:04]

Try jobs failed on following builders:
  mac_chromium_rel on tryserver.chromium

[armansito, 2014-05-03 00:01:10]

The CQ bit was checked by armansito@chromium.org

[commit-bot: I haz the power, 2014-05-03 00:02:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/armansito@chromium.org/255053002/40001

[commit-bot: I haz the power, 2014-05-03 02:25:26]

Change committed as 268009