[webkit-reviews] review granted: [Bug 48215] SVGTextElement supplemental transforms pre-multiplied but should be post-multiplied in animatedLocalTransform : [Attachment 73396] Patch

[bugzilla-daemon at webkit.org]

Dirk Schulze <krit at webkit.org> has granted Shane Stephens
<shanestephens at google.com>'s request for review:
Bug 48215: SVGTextElement supplemental transforms pre-multiplied but should be
post-multiplied in animatedLocalTransform
https://bugs.webkit.org/show_bug.cgi?id=48215

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

------- Additional Comments from Dirk Schulze <krit at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=73396&action=review

r=me

> WebCore/ChangeLog:17
> +	   Switched order of multiplication.  Note that due to a bug in 
> +	   AffineTransform's operator* implementation the order of arguments is

> +	   reversed (see http://bugs.webkit.org/show_bug.cgi?id=48031).
> +

Please add a comment that this will be fixed in a followup.