Closed
Bug 685901
Opened 14 years ago
Closed 14 years ago
Remove the nsTableOuterFrame::mInnerTableFrame and mCaptionFrame members in favor of accessing the child frame lists directly
Categories
(Core :: Layout: Tables, enhancement)
Core
Layout: Tables
Tracking
()
RESOLVED
FIXED
mozilla9
People
(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)
Details
Attachments
(1 file)
|
30.24 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
Follow-up from bug 685154 comment 9 - 12
Remove the nsTableOuterFrame::mInnerTableFrame and mCaptionFrame members in favor of accessing the child frame lists directly.
At some point we're going to need to support multiple caption frames. Hopefully this will help...
|
Assignee | |
Comment 2•14 years ago
|
||
The accessor InnerTableFrame() is mostly to avoid having to cast it to
nsTableFrame* a lot. (without Get-prefix since it can't be null)
Replace mCaptionFrame with mCaptionFrames.FirstChild(), except
where it was used for (non-)null testing for which I used
mCaptionFrames.NotEmpty() or IsEmpty() instead.
Attachment #559507 -
Flags: review?(roc)
|
|
Assignee | |
Comment 3•14 years ago
|
||
(the patch is to be applied on top of the patches in bug 685154)
Attachment #559507 -
Flags: review?(roc) → review+
|
|
Assignee | |
Comment 4•14 years ago
|
||
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
|
||
Comment 5•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
You need to log in
before you can comment on or make changes to this bug.
Description
•