Bluetooth: Disable "IsConnectable" property for ChromeOS implementation.

Bluetooth: Disable "IsConnectable" property for ChromeOS implementation.

The IsConnectable property tells if a given device can be re-connected
later once paired. If it is false, the device is not supposed to respond
to a connection attempt from the host, and since we know it, we don't
show the connect button for this kind of devices.

This fix rollbacks the IsConnectable implementation in ChromeOS and
default its value to true because some mice don't support pairing and should be possible to connect to them regardless the value of this property.

BUG=chromium:220553
TEST=Pair to Microsoft mouse 5000, disconnect and try to connect again.

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

[deymo, 2013-03-22 21:43:24]

Disbling IsConnectable().
It breaks the Microsoft 5000 mouse compatibility, and that's a regression.
Heading the change for M28 in bluetooth experimental.

[keybuk, 2013-03-25 15:58:12]

Is this a revert of a previous patch, or is this a new patch?

[deymo, 2013-03-25 16:58:04]

On 2013/03/25 15:58:12, keybuk wrote:
> Is this a revert of a previous patch, or is this a new patch?

This is a revert for part of https://codereview.chromium.org/12310048/
Only the implementation part is reverted (because we won't support that on BlueZ
4.99).

[keybuk, 2013-03-25 17:13:38]

What are the effects of the partial revert?

The connectable property will still exist - what will it's value be?

If it's value is always false, what are the effects of that on the UI?

[deymo, 2013-03-25 17:50:57]

On 2013/03/25 17:13:38, keybuk wrote:
> What are the effects of the partial revert?
> 
> The connectable property will still exist - what will it's value be?
> 
> If it's value is always false, what are the effects of that on the UI?

The IsConnectable() method exists and reads from connectable_, in
bluetooth_device.{h|cc}. The default is true, which is a safe value.

[keybuk, 2013-03-25 17:52:35]

Ok

[keybuk, 2013-03-25 17:53:12]

lgtm after adjusting the commit message to include the details discussed above

[commit-bot: I haz the power, 2013-03-25 18:30:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/deymo@chromium.org/12917023/1

[commit-bot: I haz the power, 2013-03-26 05:56:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/deymo@chromium.org/12917023/1

[commit-bot: I haz the power, 2013-03-26 06:06:17]

Change committed as 190559