bluetooth: android: removed duplicate restart of scanning.

bluetooth: android: removed duplicate restart of scanning.

This is a followup fix to commit 3a38a46. There is no need to restart the search in this callback as this will be done from native code (BluetoothChooserAndroid::SetAdapterPresence).

BUG=543060
Committed: https://crrev.com/e68e28f126ea24755a33d8b72bc1fa1719a8a5b5
Cr-Commit-Position: refs/heads/master@{#401257}

[perja, 2016-06-14 17:04:56]

perja@opera.com changed reviewers:
+ ortuno@chromium.org, scheib@chromium.org

[perja, 2016-06-14 17:04:57]

This CL fixes an issue from a previous change of mine where the discovery
session was started twice on Android (once in Java code and then again in native
code) when adapter changed state from power off to on.

[ortuno, 2016-06-14 17:22:23]

I would also remove it from BluetoothChooserAndroid::SetAdapterPresence and
instead restart the scan from
https://cs.chromium.org/chromium/src/content/browser/bluetooth/bluetooth_devi...

if (chooser_.get()) {
  chooser_->SetAdapterPresence(
      powered ? BluetoothChooser::AdapterPresence::POWERED_ON
              : BluetoothChooser::AdapterPresence::POWERED_OFF);
  OnBluetoothChooserEvent(BluetoothChooser::Event::RESCAN,
                          "" /* device_address */);  
}

This avoids having to implement the same functionality on each chooser.

[perja, 2016-06-15 15:03:23]

Makes sense to move this to the device controller. Done that now.

[ortuno, 2016-06-15 15:44:41]

On 2016/06/15 at 15:03:23, perja wrote:
> Makes sense to move this to the device controller. Done that now.

Thanks! lgtm

[perja, 2016-06-15 19:37:45]

perja@opera.com changed reviewers:
+ tedchoc@chromium.org

[perja, 2016-06-15 19:37:46]

tedchoc: OWNERS review for android files in chrome/.

[Ted C, 2016-06-15 19:55:37]

On 2016/06/15 19:37:46, perja wrote:
> tedchoc: OWNERS review for android files in chrome/.

chrome - lgtm

[perja, 2016-06-22 11:09:34]

The CQ bit was checked by perja@opera.com

[commit-bot: I haz the power, 2016-06-22 11:09:52]

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

[commit-bot: I haz the power, 2016-06-22 12:20:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-22 12:22:01]

Description was changed from

==========
bluetooth: android: removed duplicate restart of scanning.

This is a followup fix to commit 3a38a46. There is no need to restart the search
in this callback as this will be done from native code
(BluetoothChooserAndroid::SetAdapterPresence).

BUG=543060
==========

to

==========
bluetooth: android: removed duplicate restart of scanning.

This is a followup fix to commit 3a38a46. There is no need to restart the search
in this callback as this will be done from native code
(BluetoothChooserAndroid::SetAdapterPresence).

BUG=543060

Committed: https://crrev.com/e68e28f126ea24755a33d8b72bc1fa1719a8a5b5
Cr-Commit-Position: refs/heads/master@{#401257}
==========

[commit-bot: I haz the power, 2016-06-22 12:22:02]

Patchset 2 (id:??) landed as
https://crrev.com/e68e28f126ea24755a33d8b72bc1fa1719a8a5b5
Cr-Commit-Position: refs/heads/master@{#401257}