Bug 55730

Summary: UI process needs a way to tell the web process its backing store needs a full update on the next paint
Product: WebKit Reporter: Adam Roben (:aroben) <aroben>
Component: New BugsAssignee: Adam Roben (:aroben) <aroben>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 51262    
Attachments:
Description Flags
Patch andersca: review+

Adam Roben (:aroben)
Reported 2011-03-03 17:15:36 PST
UI process needs a way to tell the web process its backing store needs a full update on the next paint
Attachments
Patch (21.23 KB, patch)
2011-03-03 17:16 PST, Adam Roben (:aroben) 		andersca: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Roben (:aroben)
Comment 1 2011-03-03 17:16:05 PST
Created attachment 84659 [details] Patch
Anders Carlsson
Comment 2 2011-03-03 17:25:13 PST
Comment on attachment 84659 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=84659&action=review > Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.cpp:195 > + } Should we do this even if there's a layer tree host?
Adam Roben (:aroben)
Comment 3 2011-03-03 17:30:49 PST
Comment on attachment 84659 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=84659&action=review >> Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.cpp:195 >> + } > > Should we do this even if there's a layer tree host? Yes. The DidUpdateBackingStoreState message will tell the UI process about the layer tree host, and won't include a bitmap.
Adam Roben (:aroben)
Comment 4 2011-03-03 17:34:19 PST
Comment on attachment 84659 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=84659&action=review >>> Source/WebKit2/WebProcess/WebPage/DrawingAreaImpl.cpp:195 >>> + } >> >> Should we do this even if there's a layer tree host? > > Yes. The DidUpdateBackingStoreState message will tell the UI process about the layer tree host, and won't include a bitmap. It will also let the UI process know we've updated to match the current state (e.g., size etc.).
Adam Roben (:aroben)
Comment 5 2011-03-03 17:37:41 PST
Committed r80306: <http://trac.webkit.org/changeset/80306>
Note You need to log in before you can comment on or make changes to this bug.