Closed Bug 2036323 Opened 16 days ago Closed 9 days ago

[wpt-sync] Sync PR 59048 - Fix scroll-timeline-inactive.html

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
152 Branch
Tracking Status
firefox152 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Blocks 1 open bug, )

Details

(Whiteboard: [wptsync downstream])

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

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

Kevin Ellis <kevers@google.com> wrote:

Fix scroll-timeline-inactive.html

Problems:

  • missing timeline-scope
  • animation paused

As a result, the animation in the second sub-test was forced to have a
hold time of 0s despite supposedly having a scroll-timeline. Without
timeline-scope, the scroll-timeline was not resolved. The test
likely predated the introduction of timeline-scope rules.

Removed time-based values and animation-play-state from the animations
since these were masking the real problem in the second sub-test.
Added validation to ensure the timeline is properly resolved.

Interop 2026

Change-Id: I345f2134bea64aa1b4ef3f4d0c500ecb336bde2c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7736342
Commit-Queue: Kevin Ellis \<kevers@chromium.org>
Reviewed-by: Rune Lillesveen \<futhark@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1611480}

Component: web-platform-tests → Layout
Product: Testing → Core

CI Results

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

Total 100 tests

Status Summary

Firefox

OK : 1[GitHub] 78[Gecko-windows11-32-25h2-debug, Gecko-windows11-32-25h2-opt, Gecko-windows11-64-25h2-debug, Gecko-windows11-64-25h2-opt] 79[Gecko-android-em-14-x86_64-debug-geckoview, Gecko-android-em-14-x86_64-lite-opt-geckoview, Gecko-android-em-14-x86_64-opt-geckoview, Gecko-linux2404-64-debug, Gecko-linux2404-64-opt]
PASS: 556[Gecko-windows11-32-25h2-debug, Gecko-windows11-32-25h2-opt] 568[Gecko-android-em-14-x86_64-debug-geckoview, Gecko-android-em-14-x86_64-lite-opt-geckoview, Gecko-android-em-14-x86_64-opt-geckoview] 572[Gecko-linux2404-64-debug, Gecko-linux2404-64-opt, Gecko-windows11-64-25h2-debug, Gecko-windows11-64-25h2-opt]
FAIL: 2[GitHub] 144[Gecko-windows11-32-25h2-debug, Gecko-windows11-32-25h2-opt] 147[Gecko-windows11-64-25h2-debug, Gecko-windows11-64-25h2-opt] 148[Gecko-android-em-14-x86_64-debug-geckoview, Gecko-android-em-14-x86_64-lite-opt-geckoview, Gecko-android-em-14-x86_64-opt-geckoview, Gecko-linux2404-64-debug, Gecko-linux2404-64-opt]

Chrome

OK : 1
PASS: 2

Safari

OK : 1
PASS: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

New Tests That Don't Pass

Tests Disabled in Gecko Infrastructure

Status: NEW → RESOLVED
Closed: 9 days ago
Resolution: --- → FIXED
Target Milestone: --- → 152 Branch
You need to log in before you can comment on or make changes to this bug.