Make two promise-related tests deterministic

Make two promise-related tests deterministic

Promise rejections are implemented by a posting a trampoline task after
running microtasks[1]. A timer scheduled in the task preceding the
microtasks isn't guaranteed to run after this task, so the tests also
need to be changed to use a nested trampoline to ensure rejection has
taken place before verifying test invariants.

[1] Since https://codereview.chromium.org/1381693002/

BUG=696001
Review-Url: https://codereview.chromium.org/2813823004
Cr-Commit-Position: refs/heads/master@{#463997}
Committed: https://chromium.googlesource.com/chromium/src/+/f6a46ccf800d58cb3ff9b8c4d19eb8eee330d4d0

[Sami, 2017-04-12 10:15:50]

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

[commit-bot: I haz the power, 2017-04-12 10:16:13]

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

[Sami, 2017-04-12 10:22:07]

Description was changed from

==========
Make two promise-related tests deterministic

Promise rejections are implemented by a posting a trampoline task after
running microtasks[1]. A timer scheduled in the task preceding the
microtasks isn't guaranteed to run after this task, so the tests also
need to be changed to use a nested trampoline to ensure rejection has
taken place before verifying test invariants.

[1] Since https://codereview.chromium.org/1381693002/

BUG=696001
==========

to

==========
Make two promise-related tests deterministic

Promise rejections are implemented by a posting a trampoline task after
running microtasks[1]. A timer scheduled in the task preceding the
microtasks isn't guaranteed to run after this task, so the tests also
need to be changed to use a nested trampoline to ensure rejection has
taken place before verifying test invariants.

[1] Since https://codereview.chromium.org/1381693002/

BUG=696001
==========

[Sami, 2017-04-12 10:22:07]

skyostil@chromium.org changed reviewers:
+ jochen@chromium.org

[Sami, 2017-04-12 10:22:40]

Jochen, wanna check this since you changed the behavior (only 1½ years ago :)?

[jochen (gone - plz use gerrit), 2017-04-12 10:30:41]

lgtm

[commit-bot: I haz the power, 2017-04-12 11:18:47]

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

[commit-bot: I haz the power, 2017-04-12 11:18:48]

Dry run: This issue passed the CQ dry run.

[Sami, 2017-04-12 11:41:34]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2017-04-12 11:41:51]

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-12 11:51:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491997294665980, "parent_rev":
"acbeec26f3889dcc304c94608e2fb3bb4d2748b6", "commit_rev":
"f6a46ccf800d58cb3ff9b8c4d19eb8eee330d4d0"}

[commit-bot: I haz the power, 2017-04-12 11:52:04]

Description was changed from

==========
Make two promise-related tests deterministic

Promise rejections are implemented by a posting a trampoline task after
running microtasks[1]. A timer scheduled in the task preceding the
microtasks isn't guaranteed to run after this task, so the tests also
need to be changed to use a nested trampoline to ensure rejection has
taken place before verifying test invariants.

[1] Since https://codereview.chromium.org/1381693002/

BUG=696001
==========

to

==========
Make two promise-related tests deterministic

Promise rejections are implemented by a posting a trampoline task after
running microtasks[1]. A timer scheduled in the task preceding the
microtasks isn't guaranteed to run after this task, so the tests also
need to be changed to use a nested trampoline to ensure rejection has
taken place before verifying test invariants.

[1] Since https://codereview.chromium.org/1381693002/

BUG=696001

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

[commit-bot: I haz the power, 2017-04-12 11:52:04]

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