1529420 - Minor fixes for scroll snap web platform tests

Status: RESOLVED FIXED

(Core :: Layout: Scrolling and Overflow, enhancement, P3)

Description

[Hiroyuki Ikezoe (:hiro)]

Though initially I was going to fix all pre-existing issue in the web platform tests in the upstream directly, but I did already send a PR to the upstream, it takes more time than I thought. So I am going to fix rest of the pre-existing issue that I am awake of here.

Comment 1

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1529420 - Drop a spacer element from overflowing-snap-areas.html. r?botond!

There is no CSS declaration for #spacer. The element doesn't contribute to the
test at all.

Comment 2

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1529420 - Fix the expected scrollLeft value in vertical-rl test case. r?botond!

On RTL elements scrollLeft value is negative generally [1].
The original code relies on a Chrome bug [2], it's not yet fixed.

[1] https://github.com/w3c/csswg-drafts/issues/1354
[2] https://bugs.chromium.org/p/chromium/issues/detail?id=721759

Depends on D20588

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e054dd27efa2
Drop a spacer element from overflowing-snap-areas.html. r=botond
https://hg.mozilla.org/integration/autoland/rev/e7140d900c30
Fix the expected scrollLeft value in vertical-rl test case. r=botond

Comment 4

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/e054dd27efa2
https://hg.mozilla.org/mozilla-central/rev/e7140d900c30

Comment 5

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

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/15546 for changes under testing/web-platform/tests

Comment 6

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

Upstream PR merged