Fix BluetoothConnection to disconnect when the underlying device disconnects.

Fix BluetoothConnection to disconnect when the underlying device disconnects.

BUG=527558
TEST=unit test and manual

Committed: https://crrev.com/410dd75772d3c59593f3d78d6d3f0fbcbf6d9a78
Cr-Commit-Position: refs/heads/master@{#351238}

[Tim Song, 2015-09-24 02:38:07]

tengs@chromium.org changed reviewers:
+ sacomoto@chromium.org

[sacomoto, 2015-09-24 11:34:24]

LGTM with nit.

https://codereview.chromium.org/1359043005/diff/1/components/proximity_auth/b...
File components/proximity_auth/bluetooth_connection.cc (right):

https://codereview.chromium.org/1359043005/diff/1/components/proximity_auth/b...
components/proximity_auth/bluetooth_connection.cc:109: Disconnect();
nit: Please check if |IsConnected()| before calling |Disconnect()|, to be
consistent with the |DeviceChanged()|.

[Tim Song, 2015-09-28 22:20:38]

https://codereview.chromium.org/1359043005/diff/1/components/proximity_auth/b...
File components/proximity_auth/bluetooth_connection.cc (right):

https://codereview.chromium.org/1359043005/diff/1/components/proximity_auth/b...
components/proximity_auth/bluetooth_connection.cc:109: Disconnect();
On 2015/09/24 11:34:24, sacomoto wrote:
> nit: Please check if |IsConnected()| before calling |Disconnect()|, to be
> consistent with the |DeviceChanged()|.

Done.

[Tim Song, 2015-09-28 22:20:42]

The CQ bit was checked by tengs@chromium.org

[Tim Song, 2015-09-28 22:20:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
sacomoto@chromium.org
Link to the patchset: https://codereview.chromium.org/1359043005/#ps20001
(title: "fix")

[commit-bot: I haz the power, 2015-09-28 22:22:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1359043005/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1359043005/20001

[commit-bot: I haz the power, 2015-09-28 22:49:55]

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

[commit-bot: I haz the power, 2015-09-28 22:49:56]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Tim Song, 2015-09-29 00:39:43]

The CQ bit was checked by tengs@chromium.org

[Tim Song, 2015-09-29 00:39:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
sacomoto@chromium.org
Link to the patchset: https://codereview.chromium.org/1359043005/#ps40001
(title: "fix test")

[commit-bot: I haz the power, 2015-09-29 00:40:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1359043005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1359043005/40001

[commit-bot: I haz the power, 2015-09-29 02:24:52]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-29 02:26:49]

Patchset 3 (id:??) landed as
https://crrev.com/410dd75772d3c59593f3d78d6d3f0fbcbf6d9a78
Cr-Commit-Position: refs/heads/master@{#351238}