Bug 107059

Summary: Coordinated Graphics: Paint contents on a backing store only when RenderLayerBacking requests.
Product: WebKit Reporter: Dongseong Hwang <dongseong.hwang>
Component: Layout and RenderingAssignee: Dongseong Hwang <dongseong.hwang>
Status: NEW    
Severity: Normal CC: noam, webkit.review.bot, zeno
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 107051    
Bug Blocks:    
Attachments:
Description Flags
Patch noam: review-, noam: commit-queue-

Dongseong Hwang
Reported 2013-01-16 15:59:22 PST
Currently, Coordinated Graphics can paint contents without a setNeedsDisplay request, and sometimes Coordinated Graphics skips a setNeedsDisplay request. This patch fixes above problems. 1. CoordinatedTile constructor does not set a dirty rect. 2. CoordinatedGraphicsLayer stores pending dirty rects and flushs in updateContentBuffers() because CoordinatedGraphicsLayer creates a backing store lazily.
Attachments
Patch (4.51 KB, patch)
2013-01-16 16:03 PST, Dongseong Hwang 		noam: review-
noam: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dongseong Hwang
Comment 1 2013-01-16 16:03:24 PST
Created attachment 183051 [details] Patch
Dongseong Hwang
Comment 2 2013-01-16 16:03:55 PST
After this patch, coordinated graphics draws google gravity incorrectly, but current coordinated graphics works well by chance. This patch is right, but we need to wait Bug 107051 not to create regression.
Noam Rosenthal
Comment 3 2013-01-22 09:33:38 PST
Comment on attachment 183051 [details] Patch I don't understand how this is tested...
Note You need to log in before you can comment on or make changes to this bug.