bluetooth: Move First Device Chooser to tests

bluetooth: Move First Device Chooser to tests

Now that we have a chooser on all supported platforms we can move the
First Device Chooser fallback to content shell, the only place where it's needed.

BUG=524780
Committed: https://crrev.com/1045554135e81958e4340f9a2eb9fb088618e6d5
Cr-Commit-Position: refs/heads/master@{#403358}

[ortuno, 2016-06-29 16:40:27]

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

[commit-bot: I haz the power, 2016-06-29 16:41:33]

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

[ortuno, 2016-06-29 17:58:11]

ortuno@chromium.org changed reviewers:
+ palmer@chromium.org, peter@chromium.org

[ortuno, 2016-06-29 17:58:11]

peter: PTAL at content/shell
palmer: PTAT at t/W/p/p/m/bluetooth/web_bluetooth.mojom

[Peter Beverloo, 2016-06-29 18:45:10]

//content/shell/ lgtm

[commit-bot: I haz the power, 2016-06-29 19:10:49]

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

[commit-bot: I haz the power, 2016-06-29 19:10:50]

Dry run: This issue passed the CQ dry run.

[palmer, 2016-06-30 00:31:32]

lgtm

[ortuno, 2016-06-30 02:25:51]

ortuno@chromium.org changed reviewers:
+ scheib@chromium.org

[ortuno, 2016-06-30 02:25:52]

scheib: PTAL

[ortuno, 2016-06-30 02:26:16]

Description was changed from

==========
bluetooth: Move First Device Chooser to tests

Now that we have a chooser on all supported platforms we can move the
First Device Chooser fallback to test, the only place where it's needed.

BUG=524780
==========

to

==========
bluetooth: Move First Device Chooser to tests

Now that we have a chooser on all supported platforms we can move the
First Device Chooser fallback to content shell, the only place where it's
needed.

BUG=524780
==========

[scheib, 2016-06-30 22:18:11]

LGTM

https://codereview.chromium.org/2110813002/diff/1/content/shell/browser/layou...
File content/shell/browser/layout_test/blink_test_controller.cc (right):

https://codereview.chromium.org/2110813002/diff/1/content/shell/browser/layou...
content/shell/browser/layout_test/blink_test_controller.cc:396:
std::unique_ptr<BluetoothChooser> BlinkTestController::RunBluetoothChooser(
Out of scope of this change, but why isn't this 'GetBluetoothChooser'?

[ortuno, 2016-06-30 22:32:32]

The CQ bit was checked by ortuno@chromium.org

[ortuno, 2016-06-30 22:32:36]

https://codereview.chromium.org/2110813002/diff/1/content/shell/browser/layou...
File content/shell/browser/layout_test/blink_test_controller.cc (right):

https://codereview.chromium.org/2110813002/diff/1/content/shell/browser/layou...
content/shell/browser/layout_test/blink_test_controller.cc:396:
std::unique_ptr<BluetoothChooser> BlinkTestController::RunBluetoothChooser(
On 2016/06/30 at 22:18:11, scheib wrote:
> Out of scope of this change, but why isn't this 'GetBluetoothChooser'?

Not sure... It was added a long time ago and no one ever asked jyasskin to
change it: https://codereview.chromium.org/1286063002

[commit-bot: I haz the power, 2016-06-30 22:33:08]

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

[commit-bot: I haz the power, 2016-07-01 00:07:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-01 00:08:01]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-01 00:12:21]

Description was changed from

==========
bluetooth: Move First Device Chooser to tests

Now that we have a chooser on all supported platforms we can move the
First Device Chooser fallback to content shell, the only place where it's
needed.

BUG=524780
==========

to

==========
bluetooth: Move First Device Chooser to tests

Now that we have a chooser on all supported platforms we can move the
First Device Chooser fallback to content shell, the only place where it's
needed.

BUG=524780

Committed: https://crrev.com/1045554135e81958e4340f9a2eb9fb088618e6d5
Cr-Commit-Position: refs/heads/master@{#403358}
==========

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

Patchset 1 (id:??) landed as
https://crrev.com/1045554135e81958e4340f9a2eb9fb088618e6d5
Cr-Commit-Position: refs/heads/master@{#403358}