Closed Bug 1891429 Opened 7 months ago Closed 7 months ago

[wpt-sync] Sync PR 45707 - Introduce GetWebExposedScrollOffsets for scrollingAPIs

Categories

(Core :: CSS Parsing and Computation, task, P4)

Product:
Core
Component:
CSS Parsing and Computation
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
127 Branch
Tracking Flags:
Tracking Status
firefox127 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Blocks 1 open bug,
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Minju Kim <mkim@igalia.com> wrote:

Introduce GetWebExposedScrollOffsets for scrollingAPIs

We have encountered the issue where simply changing zoom levels
can lead to unexpected scrolling on our sample page.
This is due to updating of truncated scroll offset [1]
when FractionalScrollOffset is disabled, which led to a loss of
fractional precision, resulting in a 1px discrepancy
at the webAPI level.

To ensure compatibility with web standards while preserving the
integrity of scroll offsets, this commit introduces GetWebExposedScrollOffsets
for scrollingAPIs (e.g. Element.scrollTop, Element.scrollLeft,
window.scrollX, window.scrollY).
Also by enabling FractionalScrollOffsets, we can address
underlying issue of scroll offset truncation.

[1] https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/scroll/scrollable_area.cc;drc=43e12b162c85779c8350871dac4b319cc1874f16;l=671

Bug: 326122314

Change-Id: I070be67b0e73d925738d556ff0626e6a1119ff00
Reviewed-on: https://chromium-review.googlesource.com/5348560
WPT-Export-Revision: 2713bd4ddea9966556d879b35b5e827ef05ed48c

Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core

CI Results

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

Total 1 tests and 3 subtests

Status Summary

Firefox

OK : 1
PASS: 2
FAIL: 1

Chrome

OK : 1
PASS: 2
FAIL: 1

Safari

OK : 1
PASS: 3

Links

Gecko CI (Treeherder)
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/bd7cbe9bd041 [wpt PR 45707] - Introduce GetWebExposedScrollOffsets for scrollingAPIs, a=testonly https://hg.mozilla.org/integration/autoland/rev/dd7351703d95 [wpt PR 45707] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/bd7cbe9bd041
https://hg.mozilla.org/mozilla-central/rev/dd7351703d95

Status: NEW → RESOLVED
Closed: 7 months ago
status-firefox127: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 127 Branch
You need to log in before you can comment on or make changes to this bug.