Closed Bug 1490228 Opened 2 years ago Closed 10 months ago

[wpt-sync] Sync PR 12944 - Use document.scrollingElement to access scroll properties of the view…

Categories

(Core :: Layout, enhancement, P4)

enhancement

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Attachments

(1 file)

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

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

Frédéric Wang <fwang@igalia.com> wrote:
>  Use document.scrollingElement to access scroll properties of the view…
>  
>  …port
>  
>  See https://drafts.csswg.org/cssom-view/#dom-document-scrollingelement
>  
>  Please do not merge until someone confirms document.scrollingElement is supported by Microsoft Edge.
>  
>  https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/17744174/ seems to suggest it always returns the body.
Component: web-platform-tests → DOM
Product: Testing → Core
Component: DOM → DOM: Core & HTML
Component: DOM: Core & HTML → Layout

GitHub CI Results

wpt.fyi PR Results Base Results

Ran 8 tests and 9 subtests

Firefox

OK : 5
PASS: 11
FAIL: 1

Chrome

OK : 5
PASS: 12

Safari

OK : 5
PASS: 4
FAIL: 8

Gecko CI Results

Ran 8 tests and 8[android-em-7.0-x86_64-debug-geckoview], 9[android-em-7.0-x86_64-opt-geckoview,linux64-asan-opt,linux64-debug,linux64-opt,linux64-qr-debug,linux64-qr-opt,windows10-64-debug,windows10-64-opt,windows10-64-qr-debug,windows10-64-qr-opt,windows7-32-debug,windows7-32-opt] subtests
OK : 5
PASS : 8[android-em-7.0-x86_64-opt-geckoview], 11[linux64-asan-opt,linux64-debug,linux64-opt,linux64-qr-debug,linux64-qr-opt,windows10-64-debug,windows10-64-opt,windows10-64-qr-debug,windows10-64-qr-opt,windows7-32-debug,windows7-32-opt], 7[android-em-7.0-x86_64-debug-geckoview]
FAIL : 1[linux64-asan-opt,linux64-debug,linux64-opt,linux64-qr-debug,linux64-qr-opt,windows10-64-debug,windows10-64-opt,windows10-64-qr-debug,windows10-64-qr-opt,windows7-32-debug,windows7-32-opt], 4[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview]

Automatic update from web-platform-tests
Use document.scrollingElement to access scroll properties of the view… (#12944)

Use document.scrollingElement to access scroll properties of the viewport

See https://drafts.csswg.org/cssom-view/#dom-document-scrollingelement

Found with the following command:

find . -type f | xargs egrep 'document.body.scroll|document.documentElement.scroll'

--

wpt-commits: 251ec7054bfe6c5665b9464238dd4baffa89dbce
wpt-pr: 12944

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b2cef7ebf770
[wpt PR 12944] - Use document.scrollingElement to access scroll properties of the view…, a=testonly
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.