Open Bug 1409230 Opened 8 years ago Updated 3 years ago

IsTableCellFrame() test in nsIFrame::PeekOffset is probably bogus

Categories

(Core :: Layout, enhancement, P3)

53 Branch
enhancement

Tracking

()

Tracking Status
firefox57 --- wontfix

People

(Reporter: bzbarsky, Unassigned)

Details

We have table cells (nsMathMLmtdFrame and nsBCTableCellFrame) which are NOT IsTableCellFrame(). So this bit in nsIFrame::PeekOffset: if (aPos->mResultFrame->IsTableWrapperFrame() || aPos->mResultFrame->IsTableCellFrame()) { is probably bogus. It used to be correct for nsMathMLmtdFrame (but not nsBCTableCellFrame) back when it was a GetType() check (nsMathMLmtdFrame inherited GetType from nsTableCellFrame, but nsBCTableCellFrame overrode it).
Priority: -- → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.