bluetooth-internals: Fix flaky test

bluetooth-internals: Fix flaky test

The test was not returning the promise so the test finished without actually
executing the rest of the test.

BUG=707530
Review-Url: https://codereview.chromium.org/2829783003
Cr-Commit-Position: refs/heads/master@{#465880}
Committed: https://chromium.googlesource.com/chromium/src/+/d8adf4fbe2475512e378443279017eadfd0dc68a

[ortuno, 2017-04-20 01:26:14]

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

[commit-bot: I haz the power, 2017-04-20 01:26:43]

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

[ortuno, 2017-04-20 01:26:51]

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

[ortuno, 2017-04-20 01:26:52]

dpapad: PTAL

[dpapad, 2017-04-20 01:38:06]

LGTM

https://codereview.chromium.org/2829783003/diff/1/chrome/test/data/webui/blue...
File chrome/test/data/webui/bluetooth_internals_browsertest.js (right):

https://codereview.chromium.org/2829783003/diff/1/chrome/test/data/webui/blue...
chrome/test/data/webui/bluetooth_internals_browsertest.js:310:
TEST_F('BluetoothInternalsTest', 'Startup_BluetoothInternals',
FWIW, I am planning to disable all WebUI tests in *SAN bots. I landed this a
while ago, but got reverted, https://codereview.chromium.org/2690303003. Will
get back to it soon.

I am suspecting that even with this fix the test might still fail (meaning
timeout) on such bots.

[ortuno, 2017-04-20 01:40:32]

On 2017/04/20 at 01:38:06, dpapad wrote:
> LGTM
> 
>
https://codereview.chromium.org/2829783003/diff/1/chrome/test/data/webui/blue...
> File chrome/test/data/webui/bluetooth_internals_browsertest.js (right):
> 
>
https://codereview.chromium.org/2829783003/diff/1/chrome/test/data/webui/blue...
> chrome/test/data/webui/bluetooth_internals_browsertest.js:310:
TEST_F('BluetoothInternalsTest', 'Startup_BluetoothInternals',
> FWIW, I am planning to disable all WebUI tests in *SAN bots. I landed this a
while ago, but got reverted, https://codereview.chromium.org/2690303003. Will
get back to it soon.
> 
> I am suspecting that even with this fix the test might still fail (meaning
timeout) on such bots.

I see. With this change we will at least stop failing i.e. asserts failing, on
the other bots. Thanks!

[ortuno, 2017-04-20 01:40:35]

The CQ bit was unchecked by ortuno@chromium.org

[ortuno, 2017-04-20 01:40:38]

The CQ bit was checked by ortuno@chromium.org

[commit-bot: I haz the power, 2017-04-20 01:40:44]

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, 2017-04-20 03:10:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492652438393720, "parent_rev":
"344bc0d98bbc91ee0145bddca6b9eadc49b9ab4b", "commit_rev":
"9160f980426272e2bbdc43a9272fa2b9074386a1"}

[commit-bot: I haz the power, 2017-04-20 03:11:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492652438393720, "parent_rev":
"71a5fd1fd7389b9bdfa4cf42317bb0a52535c3a8", "commit_rev":
"d8adf4fbe2475512e378443279017eadfd0dc68a"}

[commit-bot: I haz the power, 2017-04-20 03:12:24]

Description was changed from

==========
bluetooth-internals: Fix flaky test

The test was not returning the promise so the test finished without actually
executing the rest of the test.

BUG=707530
==========

to

==========
bluetooth-internals: Fix flaky test

The test was not returning the promise so the test finished without actually
executing the rest of the test.

BUG=707530

Review-Url: https://codereview.chromium.org/2829783003
Cr-Commit-Position: refs/heads/master@{#465880}
Committed:
https://chromium.googlesource.com/chromium/src/+/d8adf4fbe2475512e37844327901...
==========

[commit-bot: I haz the power, 2017-04-20 03:12:25]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d8adf4fbe2475512e37844327901...