Bug 9497 - REGRESSION: Incomplete repainting when scrolling thumbnails on Flickr
Summary: REGRESSION: Incomplete repainting when scrolling thumbnails on Flickr
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Nobody
URL: http://flickr.com/photos/48992494@N00...
Keywords: HasReduction, Regression
Depends on:
Blocks:
 
Reported: 2006-06-18 14:19 PDT by mitz
Modified: 2006-06-22 21:02 PDT (History)
0 users

See Also:


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 Details | Formatted Diff | Diff
Cache layer repaint info right after updating position (83.94 KB, patch)
2006-06-20 09:49 PDT, mitz
hyatt: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 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.
Comment 1 mitz 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).
Comment 2 mitz 2006-06-19 23:14:38 PDT
Created attachment 8927 [details]
Cache layer repaint info right after updating position (WIP)
Comment 3 mitz 2006-06-20 09:49:36 PDT
Created attachment 8934 [details]
Cache layer repaint info right after updating position
Comment 4 Dave Hyatt 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.
Comment 5 David Kilzer (:ddkilzer) 2006-06-22 21:02:29 PDT
Committed revision 14983.