1351976 - stylo: flexbox layout is wrong under stylo

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect, P1)

Description

[Shing Lyu [:shinglyu]]

Attachment: Failing test cases

Various flexbox reftest has wrong layout, need further investigation.

Comment 1

[Bobby Holley (:bholley)]

This is a lot of tests, so bumping the priority. I'll try to find an owner soon.

Comment 2

[Bobby Holley (:bholley)]

Looks like Hiro is already working on this - thanks Hiro!

After the two dependencies are fixed, how many remaining from comment 0 are failing? If it's only a handful, we can probably close this bug and come back to them later.

Comment 3

[Hiroyuki Ikezoe (:hiro)]

Attachment: Updated failing test cases

The remaining are 27 tests (I might miss some though).

I guess a reason of most of failures in layout/reftests/flexbox/  is that we don't handle '-moz-max-content' or 'moz-min-content' properly.

I am not sure about reasons of failure tests in layout/reftests/w3c-css/submitted/align3/, need to investigate respectively.

Comment 4

[Hiroyuki Ikezoe (:hiro)]

(In reply to Hiroyuki Ikezoe (:hiro) from comment #3)
> I am not sure about reasons of failure tests in
> layout/reftests/w3c-css/submitted/align3/, need to investigate respectively.

Rest of failures in this directory use 'display: grid'.  It needs bug 1341802.
So now I think flex-box issues has been solved.