Flush all TaskScheduler<->MainLoop tasks in ~TestBrowserThreadBundle()

Flush all TaskScheduler<->MainLoop tasks in ~TestBrowserThreadBundle()

And update a bit of documentation on why some things are done a certain way.

BUG=724086
Review-Url: https://codereview.chromium.org/2893823003
Cr-Commit-Position: refs/heads/master@{#472864}
Committed: https://chromium.googlesource.com/chromium/src/+/13c37ee0b4babbdec65c0ea3c68c9d771999689c

[gab, 2017-05-18 14:55:41]

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

[commit-bot: I haz the power, 2017-05-18 14:56:26]

Dry run: CQ is trying da patch.

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

[gab, 2017-05-18 14:58:42]

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

[gab, 2017-05-18 14:58:43]

Patchset #1 (id:1) has been deleted

[gab, 2017-05-18 14:58:58]

gab@chromium.org changed reviewers:
+ fdoray@chromium.org

[gab, 2017-05-18 14:58:59]

Francois PTaL, thanks

[commit-bot: I haz the power, 2017-05-18 14:59:42]

Dry run: CQ is trying da patch.

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

[fdoray, 2017-05-18 15:27:47]

lgtm

[gab, 2017-05-18 16:15:17]

gab@chromium.org changed reviewers:
+ avi@chromium.org

[gab, 2017-05-18 16:15:18]

@avi for content rubberstamp, please CQ if LGTY

[commit-bot: I haz the power, 2017-05-18 17:36:24]

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

[commit-bot: I haz the power, 2017-05-18 17:36:25]

Dry run: This issue passed the CQ dry run.

[Avi (use Gerrit), 2017-05-18 17:39:30]

The CQ bit was checked by avi@chromium.org

[Avi (use Gerrit), 2017-05-18 17:39:31]

lgtm

stamp

[commit-bot: I haz the power, 2017-05-18 17:40:31]

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-05-18 17:47:47]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495129170781380,
"parent_rev": "a1c3f1a769e89d9462488f704e00dfd2b22ffdc9", "commit_rev":
"13c37ee0b4babbdec65c0ea3c68c9d771999689c"}

[commit-bot: I haz the power, 2017-05-18 17:48:02]

Description was changed from

==========
Flush all TaskScheduler<->MainLoop tasks in ~TestBrowserThreadBundle()

And update a bit of documentation on why some things are done a certain way.

BUG=724086
==========

to

==========
Flush all TaskScheduler<->MainLoop tasks in ~TestBrowserThreadBundle()

And update a bit of documentation on why some things are done a certain way.

BUG=724086

Review-Url: https://codereview.chromium.org/2893823003
Cr-Commit-Position: refs/heads/master@{#472864}
Committed:
https://chromium.googlesource.com/chromium/src/+/13c37ee0b4babbdec65c0ea3c68c...
==========

[commit-bot: I haz the power, 2017-05-18 17:48:06]

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