Closed Bug 685901 Opened 8 years ago Closed 8 years ago
Remove the ns
Table Outer Frame::m Inner Table Frame and m Caption Frame members in favor of accessing the child frame lists directly
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...
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)
(the patch is to be applied on top of the patches in bug 685154)
Attachment #559507 - Flags: review?(roc) → review+
Target Milestone: --- → mozilla9
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.