Bug 8449

Summary: REGRESSION: Incomplete repaint of table cell that moved
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: hyatt
Priority: P1 Keywords: Regression
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Test case
none
Patch, including change log and manual test hyatt: review+

mitz
Reported 2006-04-17 16:24:19 PDT
RenderTableSection::layoutRows() calls repaintDuringLayoutIfMoved() with an oldCellY that the cell lied about, but repaintDuringLayoutIfMoved compares it against the true m_y. Either yPos() should be used in both places or the caller should compensate for the lying. Attaching a test case.
Attachments
Test case (498 bytes, text/html)
2006-04-17 16:26 PDT, mitz 		no flags
Details
Patch, including change log and manual test (2.93 KB, patch)
2006-04-18 03:50 PDT, mitz 		hyatt: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-04-17 16:26:12 PDT
Created attachment 7785 [details] Test case Press the Test button, then force a repaint to see what it should really look like.
mitz
Comment 2 2006-04-18 03:50:34 PDT
Created attachment 7796 [details] Patch, including change log and manual test
Dave Hyatt
Comment 3 2006-04-18 14:55:02 PDT
Comment on attachment 7796 [details] Patch, including change log and manual test r=me
Dave Hyatt
Comment 4 2006-04-19 11:10:06 PDT
This bug fixes the repainting regression with textfields that occurs on google.com.
Dave Hyatt
Comment 5 2006-04-19 11:14:14 PDT
Fixed.
Note You need to log in before you can comment on or make changes to this bug.