Closed
Bug 368330
Opened 18 years ago
Closed 18 years ago
[FIX]"ASSERTION: aBandRect should be first rect within its band" with several floats
Categories
(Core :: Layout: Floats, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9alpha1
People
(Reporter: jruderman, Assigned: bzbarsky)
References
Details
(Keywords: assertion, regression, testcase)
Attachments
(2 files)
|
(deleted),
text/html
|
Details | |
|
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
###!!! ASSERTION: aBandRect should be first rect within its band: 'aBandRect->Prev() == &mBandList || aBandRect->Prev()->mBottom <= aBandRect->mTop', file /Users/admin/trunk/mozilla/layout/generic/nsSpaceManager.cpp, line 438
|
Reporter | |
Comment 1•18 years ago
|
||
This code (including the assertion) was added a few days ago in bug 270392.
Blocks: 270392
Keywords: regression
|
Assignee | |
Comment 2•18 years ago
|
||
I can't reproduce the assert on load, but I can reproduce it if I do Ctrl-+ a few times to get to a large font size. This patch fixes what I see. Does it fix what you see?
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #252954 -
Flags: superreview?(roc)
Attachment #252954 -
Flags: review?(roc)
|
|
Assignee | |
Updated•18 years ago
|
Summary: "ASSERTION: aBandRect should be first rect within its band" with several floats → [FIX]"ASSERTION: aBandRect should be first rect within its band" with several floats
Target Milestone: --- → mozilla1.9alpha
|
|
Reporter | |
Comment 3•18 years ago
|
||
Yes, your patch fixes the assertion for me :)
Attachment #252954 -
Flags: superreview?(roc)
Attachment #252954 -
Flags: superreview+
Attachment #252954 -
Flags: review?(roc)
Attachment #252954 -
Flags: review+
|
|
Assignee | |
Comment 4•18 years ago
|
||
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
|
||
Updated•18 years ago
|
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•