From 98395d00d25edfdaf92f89961774a0bc6d4cddea Mon Sep 17 00:00:00 2001 From: Florin Malita Date: Wed, 11 Oct 2017 10:13:47 -0400 Subject: [SVGDom] Fix SkSVGTransformableNode::mapToParent() The method is used for clip path resolution, and is supposed to transform the path into parent node coords -- so it should use the forward matrix, not the inverse. Change-Id: Id6eb9fbaf024b083e19f56eddb1c293becea48fc Reviewed-on: https://skia-review.googlesource.com/58280 Reviewed-by: Robert Phillips Commit-Queue: Florin Malita --- experimental/svg/model/SkSVGTransformableNode.cpp | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) (limited to 'experimental/svg') diff --git a/experimental/svg/model/SkSVGTransformableNode.cpp b/experimental/svg/model/SkSVGTransformableNode.cpp index b2ad0b1ab9..aa2a814f58 100644 --- a/experimental/svg/model/SkSVGTransformableNode.cpp +++ b/experimental/svg/model/SkSVGTransformableNode.cpp @@ -38,14 +38,6 @@ void SkSVGTransformableNode::onSetAttribute(SkSVGAttribute attr, const SkSVGValu } void SkSVGTransformableNode::mapToParent(SkPath* path) const { - if (fTransform.value().isIdentity()) { - return; - } - - SkMatrix inv; - if (!fTransform.value().invert(&inv)) { - return; - } - - path->transform(inv); + // transforms the path to parent node coordinates. + path->transform(fTransform.value()); } -- cgit v1.2.3