Bug 9497

Summary: REGRESSION: Incomplete repainting when scrolling thumbnails on Flickr
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal Keywords: HasReduction, Regression
Priority: P1    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://flickr.com/photos/48992494@N00/167175990/
Attachments:
Description Flags
Reduction
none
Cache layer repaint info right after updating position (WIP)
none
Cache layer repaint info right after updating position hyatt: review+

mitz
Reported 2006-06-18 14:19:05 PDT
When you click the left "more" arrow in any one of the thumbnail boxes on the right, the thumbnail entering from the left doesn't paint correctly, and you end up with 5 vertical strips, all taken from the right hand side of the thumbnail. The problem is similar to bug 6388 and the fix for that bug should have worked here, too. The reason it doesn't work is that absolutePosition() uses relativePositionOffsetX and relativePositionOffsetY, which access the style, so the absolutePosition() cached before layout is already affected by the modified style.
Attachments
Reduction (438 bytes, text/html)
2006-06-18 14:20 PDT, mitz 		no flags
Details
Cache layer repaint info right after updating position (WIP) (3.84 KB, patch)
2006-06-19 23:14 PDT, mitz 		no flags
DetailsFormatted DiffDiff
Cache layer repaint info right after updating position (83.94 KB, patch)
2006-06-20 09:49 PDT, mitz 		hyatt: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-06-18 14:20:20 PDT
Created attachment 8908 [details] Reduction Click the Test button, then trigger a repaint (switch to a different tab and back).
mitz
Comment 2 2006-06-19 23:14:38 PDT
Created attachment 8927 [details] Cache layer repaint info right after updating position (WIP)
mitz
Comment 3 2006-06-20 09:49:36 PDT
Created attachment 8934 [details] Cache layer repaint info right after updating position
Dave Hyatt
Comment 4 2006-06-22 15:45:54 PDT
Comment on attachment 8934 [details] Cache layer repaint info right after updating position r=me. This is a nice one.
David Kilzer (:ddkilzer)
Comment 5 2006-06-22 21:02:29 PDT
Committed revision 14983.
Note You need to log in before you can comment on or make changes to this bug.