Closed Bug 1604602 Opened 4 years ago Closed 4 years ago

[wpt-sync] Sync PR 20816 - Removed exception for setting playback rate on a scroll linked animation

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox75 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla75
Tracking Flags:
Tracking Status
firefox75 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Jordan Taylor <jortaylo@microsoft.com> wrote:

Removed exception for setting playback rate on a scroll linked animation

Initial hold time for scroll-linked animations now accounts for playback
rate.

Added test for setting playback rate on scroll linked animations

Bug: 916117
Change-Id: Ibd7e80ce60346d2f13155b13b8cc14252afc0c3a
Reviewed-on: https://chromium-review.googlesource.com/1970911
WPT-Export-Revision: 052913ce32f6e5ce6ca3467a9894085df1881e71

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 5 tests and 3 subtests

Status Summary

Firefox

OK : 5
FAIL: 40

Chrome

OK : 5
PASS: 26
FAIL: 14

Safari

OK : 5
FAIL: 40

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/scroll-animations/setting-current-time.html
Set animation current time to a value beyond effect end.: FAIL (Chrome: PASS, Safari: FAIL)
Set animation current time to a negative value.: FAIL (Chrome: PASS, Safari: FAIL)
Set animation current time to a valid value while playing.: FAIL (Chrome: PASS, Safari: FAIL)
Setting animation current time to null throws TypeError.: FAIL (Chrome: PASS, Safari: FAIL)
Setting animation.currentTime then restarting the animation should reset the current time.: FAIL (Chrome: PASS, Safari: FAIL)
Set animation current time to a valid value without playing.: FAIL (Chrome: PASS, Safari: FAIL)
Setting current time while play pending overrides the current time: FAIL (Chrome: PASS, Safari: FAIL)
Set Animation current time then scroll.: FAIL (Chrome: PASS, Safari: FAIL)
/scroll-animations/current-time-writing-modes.html
currentTime handles startScrollOffset with direction: rtl correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles direction: rtl correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles endScrollOffset with direction: rtl correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles writing-mode: vertical-rl correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles writing-mode: vertical-lr correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles endScrollOffset (inclusive case) with direction: rtl correctly: FAIL (Chrome: PASS, Safari: FAIL)
/scroll-animations/setting-playback-rate.html
Zero current time is not affected by playbackRate set while the animation is in play-pending state.: FAIL (Chrome: FAIL, Safari: FAIL)
Initial current time is scaled by playbackRate set while scroll-linked animation is in running state.: FAIL (Chrome: FAIL, Safari: FAIL)
The current time is not affected by playbackRate set while the scroll-linked animation is in play state.: FAIL (Chrome: FAIL, Safari: FAIL)
Reversing the playback rate while playing correctly impacts current time during future scrolls: FAIL (Chrome: FAIL, Safari: FAIL)
Setting the playback rate while playing preserves the current time from scrollTimeline.: FAIL (Chrome: FAIL, Safari: FAIL)
Zero initial playback rate should correctly modify initial current time.: FAIL (Chrome: FAIL, Safari: FAIL)
The playback rate affects the rate of progress of the current time when scrolling: FAIL (Chrome: FAIL, Safari: FAIL)
Setting the playback rate while play-pending preserves the set current time.: FAIL (Chrome: FAIL, Safari: FAIL)
Setting the playback rate while play-pending preserves the current time from scrollTimeline.: FAIL (Chrome: FAIL, Safari: FAIL)
Zero current time is not affected by playbackRate set while the animation is in idle state.: FAIL (Chrome: FAIL, Safari: FAIL)
The playback rate set before scroll-linked animation started playing affects the rate of progress of the current time: FAIL (Chrome: FAIL, Safari: FAIL)
Negative initial playback rate should correctly modify initial current time.: FAIL (Chrome: FAIL, Safari: FAIL)
Setting a zero playback rate while running preserves the current time: FAIL (Chrome: FAIL, Safari: FAIL)
Setting the playback rate while playing preserves the set current time.: FAIL (Chrome: FAIL, Safari: FAIL)
/scroll-animations/current-time.html
currentTime handles startScrollOffset and endScrollOffset together correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles startScrollOffset correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime calculates the correct time based on scrolled amount: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles endScrollOffset correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles startScrollOffset > endScrollOffset correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles endScrollOffset correctly (inclusive cases): FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles fill modes correctly: FAIL (Chrome: PASS, Safari: FAIL)
currentTime adjusts correctly for the timeRange: FAIL (Chrome: PASS, Safari: FAIL)
currentTime handles startScrollOffset == endScrollOffset correctly: FAIL (Chrome: PASS, Safari: FAIL)
/scroll-animations/scroll-animation.html
Animation start and current times are correct for each animation state when the animation starts playing with advanced scroller.: FAIL (Chrome: PASS, Safari: FAIL)
Animation start and current times are correct when multiple animations are attached to the same timeline.: FAIL (Chrome: PASS, Safari: FAIL)
Animation start and current times are correct for each animation state.: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5623f965cdb1
[wpt PR 20816] - Removed exception for setting playback rate on scroll linked animations, a=testonly
https://hg.mozilla.org/integration/autoland/rev/aaf6c0dbccc6
[wpt PR 20816] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/5623f965cdb1
https://hg.mozilla.org/mozilla-central/rev/aaf6c0dbccc6

Status: REOPENED → RESOLVED
Closed: 4 years ago → 4 years ago
status-firefox75: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.