1625674 - New wpt failures from PR 22501: /css/css-flexbox/flexbox-overflow-auto-001.html

Status: RESOLVED DUPLICATE of bug 1042151

(Core :: Layout: Flexbox, defect)

Description

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

The following tests have untriaged failures in the CI runs for wpt PR 22501:

New Tests That Don't Pass

/css/css-flexbox/flexbox-overflow-auto-001.html: FAIL (Chrome: PASS, Safari: FAIL)

These updates will be on mozilla-central once bug 1625562 lands.

Note: this bug is for tracking fixing the issues and is not
owned by the wpt sync bot.

This bug is linked to the relevant tests by an annotation in
https://github.com/web-platform-tests/wpt-metadata. These annotations
can be edited using the wpt interop dashboard
https://jgraham.github.io/wptdash/

If this bug is split into multiple bugs, please also update the
annotations, otherwise we are unable to track which wpt issues are
already triaged. Resolving as duplicate or closing this issue should
be cause the bot to automatically update or remove the annotation.

Comment 4

[Daniel Holbert [:dholbert]]

There are two problems here. One is a Firefox bug, and one is a test bug.

(1) [Firefox bug] All elements here are expected to show 2 scrollbars, but some of them are only showing one scrollbar. This is due to bug 1042151.

(2) [Test bug] The test is assuming the scrollbars are always on the same side of the element. But in Firefox, we place the scrollbars on the writing-mode-specific "end" side of the element, I think. Here's are some testcases showing that, with a scrollbar on the left side in Firefox:

data:text/html,<div style="background:yellow;direction:rtl;overflow:scroll;width:100px;height:100px"><div style="width:999px;height:9999px">

data:text/html,<div style="background:yellow;writing-mode:vertical-rl;overflow:scroll;width:100px;height:100px"><div style="width:999px;height:9999px">

(It looks like Safari agrees with Firefox on this, too, which is presumably why Safari "fails" this test as noted in comment 0 here.)
[EDIT: This was apparently sort of wrong. I tested the last few versions of Safari, in April 2021 -- 14, 13.1, 12.1, 11.1 -- and they don't agree with Firefox on the second testcase here.]

tonikitoo, would you mind filing a github issue or doing whatever's needed to get (2) addressed?

Comment 5

[Daniel Holbert [:dholbert]]

Category (2) (the test bug) is responsible for the Firefox (and presumably Safari) test/reference mismatches in the second row of this test.
(Especially the first and second entries in that row. The third and fourth entries are also missing a scrollbar due to Category (1) as well, but I think they'd still fail due to Category (2) if we addressed that.)