VERIFIED FIXED Bug 7916
Box repaint rect does not include the left overflow 
https://bugs.webkit.org/show_bug.cgi?id=7916
Summary Box repaint rect does not include the left overflow
mitz
Reported 2006-03-22 10:39:26 PST
See the attached test case for examples of blocks not repainting their left overflow when it they (or their content) move. In all cases, pressing the Test button should move "Lorem" together with "ipsum". I am going to attach a patch.
Attachments
Test case (1.00 KB, text/html)
2006-03-22 10:40 PST, mitz 		no flags
Details
Include left and top overflow in repaint rects (7.44 KB, patch)
2006-03-22 11:01 PST, mitz 		no flags
DetailsFormatted DiffDiff
Include left and top overflow in repaint rects (7.51 KB, patch)
2006-03-23 08:31 PST, mitz 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-03-22 10:40:24 PST
Created attachment 7232 [details] Test case
mitz
Comment 2 2006-03-22 11:01:47 PST
Created attachment 7233 [details] Include left and top overflow in repaint rects
Darin Adler
Comment 3 2006-03-23 07:24:36 PST
Comment on attachment 7233 [details] Include left and top overflow in repaint rects Seems to me that overflowRect should be passing the "include interior" boolean along to overflowWidth() and overflowHeight(), no?
mitz
Comment 4 2006-03-23 08:31:38 PST
Created attachment 7249 [details] Include left and top overflow in repaint rects Corrected the error pointed out by Darin.
Darin Adler
Comment 5 2006-03-23 09:49:06 PST
Comment on attachment 7249 [details] Include left and top overflow in repaint rects r=me
mitz
Comment 6 2006-03-27 08:05:21 PST
Eric committed the patch as r13483.
mitz
Comment 7 2006-03-31 01:11:53 PST
Verified in r13594 nightly.
Note You need to log in before you can comment on or make changes to this bug.