Bug 7916

Summary: Box repaint rect does not include the left overflow
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: VERIFIED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Test case
none
Include left and top overflow in repaint rects
none
Include left and top overflow in repaint rects darin: review+

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.