bluetooth: Include connected device in the chooser.

bluetooth: Include connected device in the chooser.

Devices already connected by other app than chome, are not advertising.
The choose adds devices returned by to:
BluetoothAdapter::RetrieveGattConnectedDevicesWithDiscoveryFilter()

Related to: https://codereview.chromium.org/2339253002

BUG=2339253002

[jlebel, 2016-11-20 19:10:11]

Description was changed from

==========
bluetooth: Include connected device in the chooser.

Devices already connected by other app than chome, are not advertising.
The choose adds devices returned by to:
BluetoothAdapter::RetrieveGattConnectedDevicesWithDiscoveryFilter()

BUG=2339253002
==========

to

==========
bluetooth: Include connected device in the chooser.

Devices already connected by other app than chome, are not advertising.
The choose adds devices returned by to:
BluetoothAdapter::RetrieveGattConnectedDevicesWithDiscoveryFilter()

Related to: https://codereview.chromium.org/2339253002

BUG=2339253002
==========

[jlebel, 2016-11-20 19:11:14]

jlebel@chromium.org changed reviewers:
+ fbeaufort@chromium.org, ortuno@chromium.org

[jlebel, 2016-11-20 19:14:13]

Hello Giovanni,

This code patch should be the last part of
https://codereview.chromium.org/2339253002
 with updating BluetoothDeviceChooserController.

[ortuno, 2016-11-21 04:48:02]

We should hold off until we implement the function on android
(http://crrev.com/2499913002). The tests for PopulateConnectedDevices are very
similar to the tests we will need for this. Rather than adding the same tests
again I would rather we implement the function on android, replace
PopulateConnectedDevices with it and fix the existing tests.

[jlebel, 2016-12-13 13:53:47]

The CQ bit was checked by jlebel@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-13 13:54:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[François Beaufort, 2016-12-13 15:28:54]

On 2016/11/21 04:48:02, ortuno wrote:
> We should hold off until we implement the function on android
> (http://crrev.com/2499913002). The tests for PopulateConnectedDevices are very
> similar to the tests we will need for this. Rather than adding the same tests
> again I would rather we implement the function on android, replace
> PopulateConnectedDevices with it and fix the existing tests.

Please update
http://forum.espruino.com/conversations/296919/?offset=25#comment13367614 when
this patch lands.

[commit-bot: I haz the power, 2016-12-13 15:45:51]

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

[commit-bot: I haz the power, 2016-12-13 15:45:51]

Dry run: This issue passed the CQ dry run.