Closed Bug 1819650 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 38773 - Avoid markers in counter-styles-at-rule tests in the same line

Categories

(Core :: Layout: Generated Content, Lists, and Counters, task, P4)

task

Tracking

()

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(Depends on 1 open bug, )

Details

(Whiteboard: [wptsync downstream])

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

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

Vitor Roriz <vitor.roriz@apple.com> wrote:

Avoid markers in counter-styles-at-rule tests in the same line

At the counter-styles at-rule tests, the markers in the tests, and the di'sv in the related references are forced to be in the same line by 'float: left' in the common style for the files.

This causes some fuzziness in WebKit tests due to subpixel differences on how the engine renders text in divs and markers in list-items.

I'd like to remove float: left from the files to avoid adding fuzziness tolerance in all these tests.

Component: web-platform-tests → Layout: Generated Content, Lists, and Counters
Product: Testing → Core
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → INVALID
Status: RESOLVED → REOPENED
Resolution: INVALID → ---

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 32 tests

Status Summary

Firefox

PASS: 32

Chrome

PASS: 26
FAIL: 6

Safari

PASS: 10
FAIL: 22

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f4daeeeb76fb [wpt PR 38773] - Avoid markers in counter-styles-at-rule tests in the same line, a=testonly https://hg.mozilla.org/integration/autoland/rev/62fcf63502a1 [wpt PR 38773] - Update wpt metadata, a=testonly
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.