| 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 Bugs | Assignee: | 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
Adam Roben (:aroben)
2011-03-03 17:15:36 PST
Created attachment 84659 [details]
Patch
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? 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. 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.). Committed r80306: <http://trac.webkit.org/changeset/80306> |