Enable layer squashing (issue 242373004)

1 view
Skip to first unread message

aba...@chromium.org

unread,
Apr 18, 2014, 1:32:00 PM4/18/14
to vol...@chromium.org, blink-...@chromium.org, bemjb+r...@chromium.org, dsin...@chromium.org, zol...@webkit.org, eae+bli...@chromium.org, leviw+re...@chromium.org, jchaffraix...@chromium.org, p...@chromium.org, oj...@chromium.org, rune+...@opera.com
Reviewers: Ian Vollick,

Description:
Enable layer squashing

This is a second attempt to enable layer squashing. Rather than enabling
this
from the Chromium side, this CL enables it on the Blink side. Once this
lands,
we can switch over to controlling this from the Chromium side so that
--disable-layer-squashing works.

Please review this at https://codereview.chromium.org/242373004/

SVN Base: svn://svn.chromium.org/blink/trunk

Affected files (+88, -90 lines):
M LayoutTests/TestExpectations
M Source/core/rendering/compositing/RenderLayerCompositor.cpp
M Source/core/rendering/svg/RenderSVGRoot.cpp


esp...@chromium.org

unread,
Apr 18, 2014, 1:37:24 PM4/18/14
to aba...@chromium.org, vol...@chromium.org, blink-...@chromium.org, bemjb+r...@chromium.org, dsin...@chromium.org, zol...@webkit.org, eae+bli...@chromium.org, leviw+re...@chromium.org, aba...@chromium.org, jchaffraix...@chromium.org, p...@chromium.org, oj...@chromium.org, rune+...@opera.com
lgtm, apparently we only get an RenderSVGRoot for the outer most <svg>, so
I'm
not sure what transforms we're respecting here.

https://codereview.chromium.org/242373004/

commi...@chromium.org

unread,
Apr 18, 2014, 1:39:22 PM4/18/14
to aba...@chromium.org, vol...@chromium.org, esp...@chromium.org, blink-...@chromium.org, bemjb+r...@chromium.org, dsin...@chromium.org, zol...@webkit.org, eae+bli...@chromium.org, leviw+re...@chromium.org, aba...@chromium.org, jchaffraix...@chromium.org, p...@chromium.org, oj...@chromium.org, rune+...@opera.com

commi...@chromium.org

unread,
Apr 18, 2014, 2:48:05 PM4/18/14
to aba...@chromium.org, vol...@chromium.org, esp...@chromium.org, blink-...@chromium.org, bemjb+r...@chromium.org, dsin...@chromium.org, zol...@webkit.org, eae+bli...@chromium.org, leviw+re...@chromium.org, aba...@chromium.org, jchaffraix...@chromium.org, p...@chromium.org, oj...@chromium.org, rune+...@opera.com
Change committed as 171962

https://codereview.chromium.org/242373004/
Reply all
Reply to author
Forward
0 new messages