Lazily allocate svg shape transforms, shrink RenderSVGShape by 44B

Source/core/rendering/svg/RenderSVGShape.cpp

 /*
  * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2008 Rob Buis <buis@kde.org>
  * Copyright (C) 2005, 2007 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2009 Google, Inc.
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) Research In Motion Limited 2010. All rights reserved.
  * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com>
  * Copyright (C) 2011 Renata Hodovan <reni@webkit.org>
  * Copyright (C) 2011 University of Szeged
@@ skipping 105 matching lines @@
 
     if (m_needsShapeUpdate || m_needsBoundariesUpdate) {
         updateShapeFromElement();
         m_needsShapeUpdate = false;
         updatePaintInvalidationBoundingBox();
         m_needsBoundariesUpdate = false;
         updateCachedBoundariesInParents = true;
     }
 
     if (m_needsTransformUpdate) {
-        m_localTransform =  toSVGGraphicsElement(element())->calculateAnimatedLocalTransform();
+        SVGGraphicsElement* graphicsElement = toSVGGraphicsElement(element());
+        m_localTransform = graphicsElement->hasAnimatedLocalTransform() ? adoptPtr(new AffineTransform(graphicsElement->calculateAnimatedLocalTransform())) : 0;

[fs, 2014/11/05 13:46:47]

It'd be nice to not reallocate the transform for each update, but I suppose a
micro-bench would be needed to even observe it - and we're already assuming that
the transform will rarely be there anyway so probably not worth the additional
code complexity at this point in time.

[f(malita), 2014/11/06 22:16:15]

+1 for avoiding unnecessary reallocations. Even if local transforms are rare for
shapes, we shouldn't penalize outliers.

[pdr., 2014/11/07 04:20:27]

Agreed and done :)

         m_needsTransformUpdate = false;
         updateCachedBoundariesInParents = true;
     }
 
     // Invalidate all resources of this client if our layout changed.
     if (everHadLayout() && selfNeedsLayout())
         SVGResourcesCache::clientLayoutChanged(this);
 
     // If our bounds changed, notify the parents.
     if (updateCachedBoundariesInParents)
@@ skipping 31 matching lines @@
         rects.append(rect);
 }
 
 bool RenderSVGShape::nodeAtFloatPoint(const HitTestRequest& request, HitTestResult& result, const FloatPoint& pointInParent, HitTestAction hitTestAction)
 {
     // We only draw in the foreground phase, so we only hit-test then.
     if (hitTestAction != HitTestForeground)
         return false;
 
     FloatPoint localPoint;
-    if (!SVGRenderSupport::transformToUserSpaceAndCheckClipping(this, m_localTransform, pointInParent, localPoint))
+    if (!SVGRenderSupport::transformToUserSpaceAndCheckClipping(this, localToParentTransform(), pointInParent, localPoint))
         return false;
 
     PointerEventsHitRules hitRules(PointerEventsHitRules::SVG_GEOMETRY_HITTESTING, request, style()->pointerEvents());
     if (nodeAtFloatPointInternal(request, localPoint, hitRules)) {
         updateHitTestResult(result, roundedLayoutPoint(localPoint));
         return true;
     }
 
     return false;
 }
@@ skipping 60 matching lines @@
 bool RenderSVGShape::hasSmoothStroke() const
 {
     const SVGRenderStyle& svgStyle = style()->svgStyle();
     return svgStyle.strokeDashArray()->isEmpty()
         && svgStyle.strokeMiterLimit() == SVGRenderStyle::initialStrokeMiterLimit()
         && svgStyle.joinStyle() == SVGRenderStyle::initialJoinStyle()
         && svgStyle.capStyle() == SVGRenderStyle::initialCapStyle();
 }
 
 }