1591441 - [wpt-sync] Sync PR 19868 - [css-lists] Add test for list-style-type:<string> with Ahem font

Status: RESOLVED FIXED

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

Description

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

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

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

Oriol Brufau <[e-mail]> wrote:

[css-lists] Add test for list-style-type:<string> with Ahem font

In the other reftests the width of the marker text is not known, so the
references have to rely on some tricks with the ::before pseudo-element.

This patch adds a test that uses the Ahem font, so that the width of the
characters is known. Then the result can be checked with a simpler
reference file.

Note it fails in LayoutNG due to the lack of scrollbar (bug 1012294),
but it passes in legacy.

Bug: 687946
Change-Id: I7fd0856e93d4e489af65b5c767ec81c109804c5c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879067
Commit-Queue: Oriol Brufau \<[e-mail]>
Reviewed-by: Manuel Rego \<[e-mail]>
Cr-Commit-Position: refs/heads/master@{#709405}

Comment 1

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

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

Comment 2

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

GitHub CI Results

wpt.fyi PR Results Base Results

Ran 1 tests

Firefox

PASS: 1

Chrome

FAIL: 1

Safari

FAIL: 1

Comment 3

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

Attachment: Bug 1591441 [wpt PR 19868] - [css-lists] Add test for list-style-type:<string> with Ahem font, a=testonly

Automatic update from web-platform-tests
[css-lists] Add test for list-style-type:<string> with Ahem font

In the other reftests the width of the marker text is not known, so the
references have to rely on some tricks with the ::before pseudo-element.

This patch adds a test that uses the Ahem font, so that the width of the
characters is known. Then the result can be checked with a simpler
reference file.

Note it fails in LayoutNG due to the lack of scrollbar (bug 1012294),
but it passes in legacy.

Bug: 687946
Change-Id: I7fd0856e93d4e489af65b5c767ec81c109804c5c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879067
Commit-Queue: Oriol Brufau <[e-mail]>
Reviewed-by: Manuel Rego <[e-mail]>
Cr-Commit-Position: refs/heads/master@{#709405}

--

wpt-commits: 52aeca2db20bd3304407bf1a1fca4e5a827db243
wpt-pr: 19868

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/77dde5467624
[wpt PR 19868] - [css-lists] Add test for list-style-type:<string> with Ahem font, a=testonly

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/77dde5467624