Closed Bug 1940132 Opened 1 month ago Closed 1 month ago

[wpt-sync] Sync PR 49930 - [carousel] Fix reserved scroll range selection edge case

Categories

(Core :: Layout: Scrolling and Overflow, task, P4)

task

Tracking

()

RESOLVED FIXED
136 Branch
Tracking Status
firefox136 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

David Awogbemila <awogbemila@chromium.org> wrote:

[carousel] Fix reserved scroll range selection edge case

It can happen that the scroll offset at which we want to select a
marker is within the "reserved" region (The portion of the scroll range
distributed[1] among scroll-markers whose snap-aligned positions fall
within that range) but the scroll targets are positioned such that the
first target is beyond the reserved region. In this case we should
rely on generic selection.

[1] https://github.com/w3c/csswg-drafts/issues/11165#issuecomment-2489115002

Bug: 383555691
Change-Id: I9243a7a0c76f287489f9db3c02c804a784dd885a
Reviewed-on: https://chromium-review.googlesource.com/6115534
WPT-Export-Revision: 084032cf52e28749138897d18dab525dd4f72026

PR 49930 applied with additional changes from upstream: 9bbcdbf23b7b1ba83b82211b46f84c488bfc98e9
Component: web-platform-tests → Layout: Scrolling and Overflow
Product: Testing → Core

CI Results

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

Total 113 tests and 1 subtests

Status Summary

Firefox

OK : 82
PASS : 291
FAIL : 83
TIMEOUT: 49
ERROR : 4
NOTRUN : 28

Chrome

OK : 106
PASS : 406
FAIL : 21
TIMEOUT: 3

Safari

OK : 51
PASS : 223
FAIL : 105
TIMEOUT: 112
ERROR : 3
NOTRUN : 43

Links

GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

New Tests That Don't Pass

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b61b5e8f8b2b [wpt PR 49930] - [carousel] Fix reserved scroll range selection edge case, a=testonly https://hg.mozilla.org/integration/autoland/rev/b4283720c79e [wpt PR 49930] - Update wpt metadata, a=testonly
Status: NEW → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in before you can comment on or make changes to this bug.