Closed Bug 1000359 Opened 11 years ago Closed 5 years ago

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

Categories

(Core :: Layout, defect)

x86_64
All
defect
Not set
critical

Tracking

()

RESOLVED DUPLICATE of bug 1639053

People

(Reporter: jruderman, Unassigned)

References

Details

(Keywords: assertion, testcase)

Attachments

(3 files)

Attached file testcase (deleted) —
Assertion failure: 2 == mFrame->StyleVisibility()->mVisible (Should only make struts for children with 'visibility:collapse'), at layout/generic/nsFlexContainerFrame.cpp:1220
Attached file stack (deleted) —
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 (deleted) —

Bug 1639053 removed the ability to reverse flex container's axes internally, so the testcases can no longer trigger assertions.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: