Closed
Bug 357544
Opened 18 years ago
Closed 18 years ago
[reflow branch] Frame(wbr)(0) frame didn't implement GetPrefWidth/GetMinWidth
Categories
(Core :: Layout: Block and Inline, defect, P2)
Tracking
()
VERIFIED
FIXED
mozilla1.9alpha3
People
(Reporter: MatsPalmgren_bugz, Assigned: dbaron)
References
Details
(Keywords: testcase, Whiteboard: [patch])
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
STEPS TO REPRODUCE
1. load the attached testcase in a debug build
ACTUAL RESULT
WARNING: Frame(wbr)(0) frame didn't implement GetPrefWidth: file nsFrame.cpp, line 2846
WARNING: Frame(wbr)(0) frame didn't implement GetMinWidth: file nsFrame.cpp, line 2832
Reporter | ||
Comment 1•18 years ago
|
||
Assignee | ||
Comment 2•18 years ago
|
||
wbr elements just get an nsFrame created for them. Given that, I should probably remove this warning -- it was there mainly to help me during development.
Assignee | ||
Comment 3•18 years ago
|
||
But I might want to leave it for a bit longer...
Assignee | ||
Updated•18 years ago
|
Whiteboard: [wbr == nsFrame]
Assignee | ||
Updated•18 years ago
|
Blocks: reflow-refactor
I get the same warning for toolbarspring
Comment 6•18 years ago
|
||
We could also create a subclass of nsLeafFrame that returns 0 for the sizes and use that for wbr...
Flags: blocking1.9?
Assignee | ||
Updated•18 years ago
|
Flags: blocking1.9? → blocking1.9+
Assignee | ||
Comment 7•18 years ago
|
||
Assignee: nobody → dbaron
Status: NEW → ASSIGNED
Assignee | ||
Updated•18 years ago
|
OS: Linux → All
Priority: -- → P2
Hardware: PC → All
Whiteboard: [wbr == nsFrame] → [patch]
Target Milestone: --- → mozilla1.9alpha3
Assignee | ||
Updated•18 years ago
|
Attachment #258500 -
Flags: superreview?(roc)
Attachment #258500 -
Flags: review?(roc)
Attachment #258500 -
Flags: superreview?(roc)
Attachment #258500 -
Flags: superreview+
Attachment #258500 -
Flags: review?(roc)
Attachment #258500 -
Flags: review+
Assignee | ||
Comment 8•18 years ago
|
||
Checked in to trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
Flags: in-testsuite-
Comment 9•17 years ago
|
||
v.fixed through proxy with ctalbert's debug build
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•