Closed Bug 363933 Opened 14 years ago Closed 14 years ago
[reflow branch] Horizontal overflow events are not dispatched anymore
See url testcase. When clicking on the "Set long string" button, an "1. overflow" text should appear at the bottom of the page. This is not happening anymore with current trunk builds. This regressed between 2006-12-07 and 2006-12-08, very likely a regression from the reflow branch landing. The overflow event is still dispatched for vertical overflows (which is what the second button on the testcase is testing), so that has not regressed.
OS: Windows XP → All
Summary: Horizontal overflow events are not dispatched anymore → [reflow branch] Horizontal overflow events are not dispatched anymore
This fixes it for me. It seems that frame->GetSize() has changed from being overflow rect to content rect by the reflow branch landing?
nsIFrame::GetSize in general is the border rect, not the content rect. But I thought I made the change you describe (to the sizing of the rect inside the scroll frame) back in bug 192767. Maybe we're thinking of different frames, though.
That said, this patch seems correct.
Comment on attachment 248788 [details] [diff] [review] fix? (In reply to comment #2) > nsIFrame::GetSize in general is the border rect, not the content rect. Right, that was what I meant.
Comment on attachment 248788 [details] [diff] [review] fix? r+sr=dbaron
Checked in to trunk at 2006-12-19 17:17 PST -> FIXED
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.