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

Status: RESOLVED FIXED

(Core :: Layout, enhancement, P4)

Description

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

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 <[e-mail]> 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.

Comment 1

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=f2ff23b209a2239ba7c63abdc411b628e0031465

Comment 2

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=a71269f1efadea81f2c67f5965db05fd256536f6

Comment 3

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

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]

Comment 4

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Attachment: Bug 1490228 [wpt PR 12944] - Use document.scrollingElement to access scroll properties of the view…, a=testonly

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

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b2cef7ebf770
[wpt PR 12944] - Use document.scrollingElement to access scroll properties of the view…, a=testonly

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/b2cef7ebf770