DescriptionResolve promises synchronously sometimes
https://codereview.chromium.org/2615253002 changed
~PlayStateUpdateScope() to resolve promises asynchronously in order to
avoid script execution inside a forbidden scope. However, this causes
the promises to behave differently from expected in some cases,
demonstrated in part by a crash in the interfaces/Animation/cancel.html
test due to failing a DCHECK.
This patch modifies the promise resolution code to first check whether
we are in a forbidden scope, and only resolve the promise asynchronously
if so.
BUG=708887
Review-Url: https://codereview.chromium.org/2808013002
Cr-Commit-Position: refs/heads/master@{#463907}
Committed: https://chromium.googlesource.com/chromium/src/+/220578cfcb90370db3b268c6e59c1c4417a60708
Patch Set 1 #
Messages
Total messages: 15 (9 generated)
|