Open Bug 1000359 Opened 6 years ago Updated 3 years ago

Assertion failure: "Should only make struts for children with 'visibility:collapse'" with flexbox

Categories

(Core :: Layout, defect, critical)

x86_64
All
defect
Not set
critical

Tracking

()

People

(Reporter: jruderman, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: assertion, testcase)

Attachments

(3 files)

Attached file testcase
Assertion failure: 2 == mFrame->StyleVisibility()->mVisible (Should only make struts for children with 'visibility:collapse'), at layout/generic/nsFlexContainerFrame.cpp:1220
Attached file stack
I can reproduce this in my Linux debug build.

Marking as depending on bug 783470, which added the "visibility:collapse" behavior for flexbox.
Depends on: 783470
OS: Mac OS X → All
Yeah, this is a regression from bug 983427.

We build the strut-list by walking over our flex lines & their flex items, which may be in reverse order with respect to the frame list. (after bug 983427)  So the strut indexes may be indexes with respect to the *end* of our child list (walking in reverse). But when we use them, in GenerateFlexLines, we treat them as if they're indexes from the start.
Blocks: 983427
Attached file less weird testcase
You need to log in before you can comment on or make changes to this bug.