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...
Created attachment 559507 [details] [diff] [review]
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.
(the patch is to be applied on top of the patches in bug 685154)