Lazily allocate svg shape transforms, shrink RenderSVGShape by 44B

Source/core/svg/SVGGraphicsElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006 Rob Buis <buis@kde.org>
  * Copyright (C) 2014 Google, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 104 matching lines @@
 PassRefPtr<SVGMatrixTearOff> SVGGraphicsElement::getCTMFromJavascript()
 {
     return SVGMatrixTearOff::create(getCTM());
 }
 
 PassRefPtr<SVGMatrixTearOff> SVGGraphicsElement::getScreenCTMFromJavascript()
 {
     return SVGMatrixTearOff::create(getScreenCTM());
 }
 
+bool SVGGraphicsElement::hasAnimatedLocalTransform() const
+{
+    RenderStyle* style = renderer() ? renderer()->style() : 0;
+    return (style && style->hasTransform()) || !m_transform->currentValue()->isEmpty() || hasSVGRareData();

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

Maybe there ought to be a comment here pointing to the method below - which
would serve to explain why thi predicate is correct (hasSVGRareData() will not
be directly obvious for instance).

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

Excellent point. Done

+}
+
 AffineTransform SVGGraphicsElement::calculateAnimatedLocalTransform() const
 {
     AffineTransform matrix;
     RenderStyle* style = renderer() ? renderer()->style() : 0;
 
     // If CSS property was set, use that, otherwise fallback to attribute (if set).
     if (style && style->hasTransform()) {
         TransformationMatrix transform;
         float zoom = style->effectiveZoom();
 
@@ skipping 122 matching lines @@
     return new RenderSVGPath(this);
 }
 
 void SVGGraphicsElement::toClipPath(Path& path)
 {
     updatePathFromGraphicsElement(this, path);
     path.transform(calculateAnimatedLocalTransform());
 }
 
 }