Bug 99529 – GraphicsLayer visible rect computation needs to use the current animating transform

• Component: New Bugs
• Status: RESOLVED FIXED

[Simon Fraser (smfr)]

GraphicsLayer visible rect computation needs to use the current animating transform

[Simon Fraser (smfr)]

Created attachment 169067 [details]
Patch

[Dean Jackson]

Comment on attachment 169067 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=169067&action=review

> Source/WebCore/platform/graphics/GraphicsLayerClient.h:75
> +    virtual void currentTransform(const GraphicsLayer*, TransformationMatrix&) const { }

It's a bit strange that this method is called currentTransform like the existing method on RenderLayer, but modifies its parameter rather than providing a return value.

> Source/WebCore/rendering/RenderLayer.h:598
> -    TransformationMatrix currentTransform() const;
> +    TransformationMatrix currentTransform(RenderStyle::ApplyTransformOrigin = RenderStyle::IncludeTransformOrigin) const;

This is the existing one (in another class)

> Source/WebCore/rendering/RenderLayerBacking.h:151
> +    virtual void currentTransform(const GraphicsLayer*, TransformationMatrix&) const;

This is the new one.

> LayoutTests/compositing/visible-rect/animated.html:26
> +        @-webkit-keyframes move {
> +            from { -webkit-transform: translateX(-100px); }
> +            to   { -webkit-transform: translateX(300px); }
> +        }

I'm not sure how this test is exercising the new code. You don't have anything here that setting transform origin.

[Simon Fraser (smfr)]

http://trac.webkit.org/changeset/131626