Closed Bug 393671 Opened 17 years ago Closed 17 years ago

[FIX]"ASSERTION: expected the float to be a child of the outer CB" and more with float, block-in-inline

Categories

(Core :: Layout, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla1.9alpha8

People

(Reporter: jruderman, Assigned: bzbarsky)

References

Details

(Keywords: assertion, testcase)

Attachments

(2 files)

Attached file testcase
Loading the testcase triggers a pair of assertions:

###!!! ASSERTION: expected the float to be a child of the outer CB: 'outOfFlowFrame->GetParent() == aOuterState.mFloatedItems.containingBlock', file /Users/jruderman/trunk/mozilla/layout/base/nsCSSFrameConstructor.cpp, line 1587

###!!! ASSERTION: float wasn't in the outer state float list: 'Not Reached', file /Users/jruderman/trunk/mozilla/layout/base/nsCSSFrameConstructor.cpp, line 1592

Applying the patches in bug 393649 and bug 393517 doesn't make these assertions go away.
Attached patch FixSplinter Review
This goes on top of those two patches.  The problem here is that we need to move the floats to the "final" reflow state, and without this patch we're not doing it.

The reftests fail without this patch: the text in the float doesn't show up.
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #278255 - Flags: superreview?(roc)
Attachment #278255 - Flags: review?(roc)
OS: Mac OS X → All
Priority: -- → P1
Hardware: PC → All
Summary: "ASSERTION: expected the float to be a child of the outer CB" and more with float, block-in-inline → [FIX]"ASSERTION: expected the float to be a child of the outer CB" and more with float, block-in-inline
Target Milestone: --- → mozilla1.9 M8
Attachment #278255 - Flags: superreview?(roc)
Attachment #278255 - Flags: superreview+
Attachment #278255 - Flags: review?(roc)
Attachment #278255 - Flags: review+
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Blocks: 390425
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: