Handle change to pending state when resolving the ready promise #22441
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The ready promise depends on the pending state, which in turn can be
impacted by a pending style change. Checking the pending state prior
to checking the ready promise fixes the problem. A few tweaks were
needed to the test itself. If we create a ready promise, we need to
wait for it to resolve. Otherwise, an error is potentially triggered
(race condition) during tear down of the test as it will cancel a
pending animation, which in turn, rejects the ready promise. Any
unexpected promise reject triggers a harness error.
Bug: 1064640
Change-Id: I25191dd26206368e497638ce9f73984b2e61cad4
Reviewed-on: https://chromium-review.googlesource.com/2118183
WPT-Export-Revision: e453aa387d3010fd8800a44a1ea509dac2121878