[webkit-reviews] review canceled: [Bug 88102] RenderLayerCompositor::computeCompositingRequirements takes very long time in GMail with scroll layer promoted : [Attachment 145324] Patch

[bugzilla-daemon at webkit.org]

W. James MacLean <wjmaclean at chromium.org> has canceled W. James MacLean
<wjmaclean at chromium.org>'s request for review:
Bug 88102: RenderLayerCompositor::computeCompositingRequirements takes very
long time in GMail with scroll layer promoted
https://bugs.webkit.org/show_bug.cgi?id=88102

Attachment 145324: Patch
https://bugs.webkit.org/attachment.cgi?id=145324&action=review

------- Additional Comments from W. James MacLean <wjmaclean at chromium.org>
This is a patch for some general discussion about the time it takes per scroll
event.

This patch, when applied and the GMail message list is promoted to it's own
compositor via translateZ(0), reduces latency between scroll events by roughly
18%, and does not seem to cause any malfunction in Chrome (although it hasn't
been tried on other ports). I was motivated to try it when I noticed that
computeCompositingRequirements() always seemed to come back with layersChanged
== false when called via RenderLayer::scrollTo().

This bug is related to
http://code.google.com/p/chromium/issues/detail?id=130115