Closed Bug 1451026 Opened 6 years ago Closed 6 years ago

[wpt-sync] Sync PR 10281 - Web Animations: Fix bug with Document::getAnimations

Categories

(Core :: DOM: Animation, enhancement, P4)

Product:
Core
Component:
DOM: Animation
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla61
Tracking Flags:
Tracking Status
firefox61 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream error]) 

Sync web-platform-tests PR 10281 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/w3c/web-platform-tests/pull/10281
Details from upstream follow.

Stephen McGruer <smcgruer@chromium.org> wrote:
>  Web Animations: Fix bug with Document::getAnimations
>  
>  Per spec, we should exclude effects that do not target an element in
>  the called document:
>  
>  https://drafts.csswg.org/web-animations-1/#dom-document-getanimations
>  
>  This CL also fixes an issue with the getAnimations.html test. It
>  previously presumed that GC would run between tests to collect animations
>  from previous tests, but that isn't required to happen. To avoid that
>  sort of issue, the tests were changed to explicitly finish their animations.
>  
>  Bug: 828424
>  Change-Id: I41405d82184b17c1185931e34735a5f946573844
>  
>  Reviewed-on: https://chromium-review.googlesource.com/992812
>  WPT-Export-Revision: 2ae062a929901399a831275571ca43b3c2f5e482
Component: web-platform-tests → DOM: Animation
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d8d73c38711f
[wpt PR 10281] - Web Animations: Fix bug with Document::getAnimations, a=testonly
https://hg.mozilla.org/mozilla-central/rev/d8d73c38711f
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.