284545 2024-12-12 06:52:04 -0800 [scroll-animations] WPT test scroll-animations/scroll-timelines/scroll-animation-effect-phases.tentative.html has failures 2025-02-21 07:04:05 -0800 1 1 1 Unclassified WebKit Animations WebKit Nightly Build Unspecified Unspecified RESOLVED FIXED https://github.com/web-platform-tests/wpt/pull/50858 InRadar P2 Normal --- 284541 1 graouts graouts graouts webkit-bug-importer oldest_to_newest 2081165 0 graouts 2024-12-12 06:52:04 -0800 The WPT test scroll-animations/scroll-timelines/scroll-animation-effect-phases.html has one failure: FAIL Playback rate affects whether active phase boundary is inclusive. assert_not_equals: Animation effect is in active phase when current time is 100%. got disallowed value null 2081166 1 webkit-bug-importer 2024-12-12 06:52:13 -0800 <rdar://problem/141356963> 2095145 2 graouts 2025-02-17 06:29:41 -0800 We fail the "Playback rate affects whether active phase boundary is inclusive" subtest in `scroll-animations/scroll-timelines/scroll-animation-effect-phases.tentative.html`. There are two issues here. The first issue is a faulty assertion function where `assert_phase` does not deal with `currentTime = 100%` being in the "active" phase for animations associated with a progress-based timeline. After fixing this we hit a debug ASSERT in WebKit though. Under `setBindingsTimeline()` at the very end of this test, when we change to the document timeline, we end up in a bad state where after calling `setCurrentTime()` the hold time is 0s but the start time is 100%. After that we set the hold time to the unresolved value since the start time is resolved. Then we're in a state where all the timing values are time-based except for the start time which remains progress-based. The test attempts to resolve this by setting `animation.currentTime = 0` right after, but we hit the debug assertion before being able to set that value. 2095147 3 graouts 2025-02-17 06:29:51 -0800 *** Bug 287793 has been marked as a duplicate of this bug. *** 2096678 4 graouts 2025-02-21 04:35:01 -0800 Pull request: https://github.com/WebKit/WebKit/pull/41055 2096679 5 graouts 2025-02-21 04:35:29 -0800 Submitted web-platform-tests pull request: https://github.com/web-platform-tests/wpt/pull/50858 2096697 6 ews-feeder 2025-02-21 07:04:03 -0800 Committed 290787@main (03eccf44ee0d): <https://commits.webkit.org/290787@main> Reviewed commits have been landed. Closing PR #41055 and removing active labels.