Don’t smash globals used by the test helper itself.

Don’t smash globals used by the test helper itself.

Since https://codereview.chromium.org/366103005 the promise tests null out
various globals, to ensure the promise implementation doesn’t itself rely
on functions patchable by monkeys.

Unfortunately, doing so breaks test assertion failures which rely on
those globals.

This isn’t the ideal solution, but does improve the current state.

R=littledan@chromium.org,domenic@chromium.org
LOG=N
BUG=

Committed: https://crrev.com/ab9898980eeeaedc99c8f539bac9e72fad8383a4
Cr-Commit-Position: refs/heads/master@{#30922}

[stefan.penner, 2015-09-24 21:17:43]

Don’t smash globals used by the test helper itself.

Since https://codereview.chromium.org/366103005 the promise tests null out
various globals, to ensure the promise implementation doesn’t itself rely
on functions patchable by monkeys.

Unfortunately, doing so breaks test assertion failures which rely on
those globals.

This isn’t the ideal solution, but does improve the current state.

R=littledan@chromium.org,domenic@chromium.org
LOG=N
BUG=

[Dan Ehrenberg, 2015-09-24 21:23:20]

The CQ bit was checked by littledan@chromium.org

[Dan Ehrenberg, 2015-09-24 21:23:20]

lgtm

[commit-bot: I haz the power, 2015-09-24 21:23:34]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1370583002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1370583002/1

[commit-bot: I haz the power, 2015-09-24 21:47:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-09-24 21:47:51]

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