Summary: | [iOS WebKit2] RemoteLayerTreeDrawingArea should force CA to garbage collect surfaces. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Kling <kling> | ||||
Component: | Layout and Rendering | Assignee: | Andreas Kling <kling> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | kling, thorton | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Andreas Kling
2014-04-24 18:12:34 PDT
Created attachment 230125 [details]
Patch
Comment on attachment 230125 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=230125&action=review > Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:288 > + [CATransaction begin]; I wonder if it's better to do this in didUpdate() instead; we're more likely to actually do something more immediately useful then, because by that point we would (ideally but not always) have pushed the new front surfaces to CA. (In reply to comment #2) > (From update of attachment 230125 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=230125&action=review > > > Source/WebKit2/WebProcess/WebPage/mac/RemoteLayerTreeDrawingArea.mm:288 > > + [CATransaction begin]; > > I wonder if it's better to do this in didUpdate() instead; we're more likely to actually do something more immediately useful then, because by that point we would (ideally but not always) have pushed the new front surfaces to CA. All right. Will move the "logic" there instead. Committed r167792: <http://trac.webkit.org/changeset/167792> |