Closed
Bug 379217
Opened 18 years ago
Closed 17 years ago
"ASSERTION: aFrame is already associated with a region" with float in button
Categories
(Core :: Layout: Floats, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: jruderman, Unassigned)
References
Details
(Keywords: assertion, testcase)
Attachments
(1 file, 1 obsolete file)
(deleted),
application/xhtml+xml
|
Details |
###!!! ASSERTION: aFrame is already associated with a region: '!GetFrameInfoFor(aFrame)', file /Users/jruderman/trunk/mozilla/layout/generic/nsSpaceManager.cpp, line 866
Comment 1•18 years ago
|
||
It looks like we reflow the float on the first line, then push the float placeholder on the overflow list of the inline in question (together with the text "bbb", etc). So we end up reflowing the float _again_ on the second line, and that's when we assert...
Note also the warning about the float cache not matching the actual float list that ensues.
Flags: blocking1.9?
Reporter | ||
Comment 2•18 years ago
|
||
Is the rendering correct? It seems strange to me that the seemingly adjacent "aaa" and "bbb" are split, but I'm not really familiar with the rules for floats.
Comment 3•18 years ago
|
||
I don't think we should be splitting the text there, no.
New textframe doesn't split there. That also means it suppresses this bug, although the underlying issue is probably still around.
Depends on: 333659
Updated•18 years ago
|
Flags: blocking1.9? → blocking1.9-
Reporter | ||
Comment 5•17 years ago
|
||
WFM with this testcase on trunk (since new textframe has landed). It would be nice if someone would create a new testcase that still demonstrates the bug.
Reporter | ||
Comment 6•17 years ago
|
||
I see the assertion on Mac trunk with this testcase. Does it demonstrate the same bug or a different bug?
Attachment #263216 -
Attachment is obsolete: true
Reporter | ||
Comment 7•17 years ago
|
||
WFM with both testcases now.
roc, is the underlying issue you mentioned in comment 4 still around?
I don't know, but if testcase2 is WFY, then I have no reason to believe it is.
Reporter | ||
Comment 9•17 years ago
|
||
Ok.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WORKSFORME
Updated•17 years ago
|
Flags: in-testsuite?
Reporter | ||
Comment 10•17 years ago
|
||
I checked in both testcases as crashtests.
Flags: in-testsuite? → in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•