1592071 - [wpt-sync] Sync PR 19955 - [scroll-snap] Make tests independent of scrollbar

Status: RESOLVED FIXED

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

Description

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

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

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

Majid Valipour <[e-mail]> wrote:

[scroll-snap] Make tests independent of scrollbar

Multiple scroll snap wpt tests use scrollbar-width:none css property
to hide scrollbars. This property is only supported in Gecko.

This CL adds the legacy webkit prefixed property [1] to have the same
functionality. This addition allows 7 tests to pass in Chrome
(and potentially in Safari).

[1] ::-webkit-scrollar { display: none; }

Fixed: 1003055
Change-Id: Iefe5f5071d23cd40bab71a9d0e360b62e10e9c73
Reviewed-on: https://chromium-review.googlesource.com/1884816
WPT-Export-Revision: 6770c64bb3e9307df2fa1d7a9f7aa5123b587d6c

Comment 1

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

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

Comment 2

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

GitHub CI Results

wpt.fyi PR Results Base Results

Ran 12 tests

Firefox

PASS: 6
FAIL: 6

Chrome

PASS: 8
FAIL: 4

Safari

FAIL: 12

Gecko CI Results

Ran 12 tests
PASS : 4[linux64-opt,linux64-qr-debug,linux64-qr-opt,windows10-64-qr-opt], 6[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview,linux64-asan-opt,linux64-debug,windows10-64-debug,windows10-64-opt,windows10-64-qr-debug,windows7-32-debug,windows7-32-opt]
FAIL : 4[linux64-opt,linux64-qr-debug,linux64-qr-opt,windows10-64-qr-opt], 6[android-em-7.0-x86_64-debug-geckoview,android-em-7.0-x86_64-opt-geckoview,linux64-asan-opt,linux64-debug,windows10-64-debug,windows10-64-opt,windows10-64-qr-debug,windows7-32-debug,windows7-32-opt]
ERROR : 4[linux64-opt,linux64-qr-debug,linux64-qr-opt,windows10-64-qr-opt]

Existing tests that now have a worse result

/css/css-scroll-snap/scroll-target-align-001.html: ERROR[linux64-opt,linux64-qr-opt]
/css/css-scroll-snap/scroll-target-margin-001.html: ERROR[linux64-opt,linux64-qr-opt]
/css/css-scroll-snap/scroll-target-padding-001.html: ERROR[linux64-opt,linux64-qr-opt]
/css/css-scroll-snap/scroll-target-snap-001.html: ERROR[linux64-opt,linux64-qr-opt]

Comment 3

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

Attachment: Bug 1592071 [wpt PR 19955] - [scroll-snap] Make several wpt tests independent of scrollbar-width, a=testonly

Automatic update from web-platform-tests
[scroll-snap] Make several wpt tests independent of scrollbar-width

Multiple scroll snap wpt tests use scrollbar-width:none css property
to hide scrollbars. This property is only supported in Gecko.

This CL adds the legacy webkit prefixed property [1] to enable the same
functionality. This addition allows 8 existing tests to pass in Chrome (and
potentially in Safari).

[1] ::-webkit-scrollar { display: none; }

Fixed: 1003055
Change-Id: Iefe5f5071d23cd40bab71a9d0e360b62e10e9c73
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1884816
Auto-Submit: Majid Valipour <[e-mail]>
Reviewed-by: Stephen McGruer <[e-mail]>
Commit-Queue: Majid Valipour <[e-mail]>
Cr-Commit-Position: refs/heads/master@{#711384}

--

wpt-commits: dc048c67d786f646ab659ea050183989ff3e8bfb
wpt-pr: 19955

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/16b52f0c8183
[wpt PR 19955] - [scroll-snap] Make several wpt tests independent of scrollbar-width, a=testonly

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/16b52f0c8183