[webkit-reviews] review granted: [Bug 89216] RenderView layer is marked as fixed position container in the scrolling tree if page scale != 1 : [Attachment 148580] Patch

[bugzilla-daemon at webkit.org]

Simon Fraser (smfr) <simon.fraser at apple.com> has granted Sami Kyöstilä
<skyostil at chromium.org>'s request for review:
Bug 89216: RenderView layer is marked as fixed position container in the
scrolling tree if page scale != 1
https://bugs.webkit.org/show_bug.cgi?id=89216

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

------- Additional Comments from Simon Fraser (smfr) <simon.fraser at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=148580&action=review


> Source/WebCore/ChangeLog:8
> +	   Render layers with CSS transforms should become containers for
anyfixed

any/fixed

> Source/WebCore/rendering/RenderLayerBacking.cpp:455
> +	       bool isContainer = m_owningLayer->hasTransform() &&
!renderer()->isRenderView();

We also use isRootLayer() for the renderview's layer.