Bug 132160 – [iOS WebKit2] RemoteLayerTreeDrawingArea should force CA to garbage collect surfaces.

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Andreas Kling]

Add gratuitous flush for <rdar://problem/16110687>

[Andreas Kling]

Created attachment 230125 [details]
Patch

[Tim Horton]

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.

[Andreas Kling]

(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.

[Andreas Kling]

Committed r167792: <http://trac.webkit.org/changeset/167792>