Bug 11045

Summary: failing manual-tests/left-overflow-repaint.html
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: ddkilzer
Priority: P2 Keywords: HasReduction
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Reduction none

mitz
Reported 2006-09-26 09:15:04 PDT
Test 3 in manual-tests/left-overflow-repaint.html is failing to repaint correctly in TOT (it also fails differently in shipping WebKit, but that was fixed in r13483, bug 7916). The regression happened sometime between r14652 and r14663, probably in one of r14656 (bug 8899), r14657 (bug 8910) and r14659 (bug 9012).
Attachments
Reduction (248 bytes, text/html)
2006-09-26 10:30 PDT, mitz 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-09-26 09:54:43 PDT
Caused by r14659. The height change results in the section being marked as needing layout, which suppresses the repainting of the moving cell in RenderTableSection::layoutRows().
mitz
Comment 2 2006-09-26 10:30:54 PDT
Created attachment 10782 [details] Reduction Okay, I think this shows the real problem: tables' repaint rects don't account for their cells' overflow. In this test case, "Lorem ipsum" should appear in the blue rect.
mitz
Comment 3 2006-09-26 23:01:18 PDT
Downgrading to P2 since this isn't a regression from the shipping version of WebKit. In fact, given the root cause of the bug, I'll just remove all mention of "regression". Sorry about the false alarm, Dave :)
mitz
Comment 4 2006-09-28 10:09:26 PDT
*** This bug has been marked as a duplicate of 10898 ***
Note You need to log in before you can comment on or make changes to this bug.