[webkit-reviews] review granted: [Bug 84807] [chromium] Don't call calculateRenderPasses in CCLayerTreeHostImpl::scrollBegin() : [Attachment 138696] Patch

[bugzilla-daemon at webkit.org]

James Robinson <jamesr at chromium.org> has granted Adrienne Walker
<enne at google.com>'s request for review:
Bug 84807: [chromium] Don't call calculateRenderPasses in
CCLayerTreeHostImpl::scrollBegin()
https://bugs.webkit.org/show_bug.cgi?id=84807

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

------- Additional Comments from James Robinson <jamesr at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=138696&action=review


Much better R=me

> Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostImpl.cpp:266
> +    ASSERT(passes.isEmpty());

can we make passes a local variable again and just have the
renderSurfaceLayerList be a member variable, or is that teh crazy?