Make 'transform' a presentation attribute on SVG elements

third_party/WebKit/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.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  */
 
 #include "core/svg/SVGGraphicsElement.h"
 
 #include "core/SVGNames.h"
-#include "core/layout/svg/LayoutSVGPath.h"
+#include "core/dom/StyleChangeReason.h"
+#include "core/layout/LayoutObject.h"
 #include "core/svg/SVGElementRareData.h"
 #include "core/svg/SVGMatrixTearOff.h"
 #include "core/svg/SVGRectTearOff.h"
 #include "platform/transforms/AffineTransform.h"
 
 namespace blink {
 
 SVGGraphicsElement::SVGGraphicsElement(const QualifiedName& tagName,
                                        Document& document,
                                        ConstructionType constructionType)
     : SVGElement(tagName, document, constructionType),
       SVGTests(this),
-      m_transform(
-          SVGAnimatedTransformList::create(this, SVGNames::transformAttr)) {
+      m_transform(SVGAnimatedTransformList::create(this,
+                                                   SVGNames::transformAttr,
+                                                   CSSPropertyTransform)) {
   addToPropertyMap(m_transform);
 }
 
 SVGGraphicsElement::~SVGGraphicsElement() {}
 
 DEFINE_TRACE(SVGGraphicsElement) {
   visitor->trace(m_transform);
   SVGElement::trace(visitor);
   SVGTests::trace(visitor);
 }
@@ skipping 51 matching lines @@
 }
 
 SVGMatrixTearOff* SVGGraphicsElement::getCTMFromJavascript() {
   return SVGMatrixTearOff::create(getCTM());
 }
 
 SVGMatrixTearOff* SVGGraphicsElement::getScreenCTMFromJavascript() {
   return SVGMatrixTearOff::create(getScreenCTM());
 }
 
+void SVGGraphicsElement::collectStyleForPresentationAttribute(
+    const QualifiedName& name,
+    const AtomicString& value,
+    MutableStylePropertySet* style) {
+  if (name == SVGNames::transformAttr) {
+    addPropertyToPresentationAttributeStyle(
+        style, CSSPropertyTransform, m_transform->currentValue()->cssValue());
+    return;
+  }
+  SVGElement::collectStyleForPresentationAttribute(name, value, style);
+}
+
 bool SVGGraphicsElement::hasAnimatedLocalTransform() const {
-  const ComputedStyle* style =
-      layoutObject() ? layoutObject()->style() : nullptr;
-
   // Each of these is used in
   // SVGGraphicsElement::calculateAnimatedLocalTransform to create an animated
   // local transform.
-  return (style && style->hasTransform()) ||
-         !m_transform->currentValue()->isEmpty() || hasSVGRareData();
+  return hasTransform() || hasSVGRareData();
 }
 
 AffineTransform SVGGraphicsElement::calculateAnimatedLocalTransform() const {
-  AffineTransform matrix;
-  const ComputedStyle* style =
-      layoutObject() ? layoutObject()->style() : nullptr;
-
-  // If CSS property was set, use that, otherwise fallback to attribute (if
-  // set).
-  if (style && style->hasTransform()) {
-    TransformationMatrix transform;
-    float zoom = style->effectiveZoom();
-
-    // SVGTextElements need special handling for the text positioning code.
-    if (isSVGTextElement(this)) {
-      // Do not take into account SVG's zoom rules, transform-origin, or
-      // percentage values.
-      style->applyTransform(
-          transform, LayoutSize(0, 0), ComputedStyle::ExcludeTransformOrigin,
-          ComputedStyle::IncludeMotionPath,
-          ComputedStyle::IncludeIndependentTransformProperties);
-    } else {
-      // CSS transforms operate with pre-scaled lengths. To make this work with
-      // SVG (which applies the zoom factor globally, at the root level) we
-      //
-      //   * pre-scale the bounding box (to bring it into the same space as the
-      //     other CSS values)
-      //   * invert the zoom factor (to effectively compute the CSS transform
-      //     under a 1.0 zoom)
-      //
-      // Note: objectBoundingBox is an emptyRect for elements like pattern or
-      // clipPath.  See the "Object bounding box units" section of
-      // http://dev.w3.org/csswg/css3-transforms/
-      if (zoom != 1) {
-        FloatRect scaledBBox = layoutObject()->objectBoundingBox();
-        scaledBBox.scale(zoom);
-        transform.scale(1 / zoom);
-        style->applyTransform(
-            transform, scaledBBox, ComputedStyle::IncludeTransformOrigin,
-            ComputedStyle::IncludeMotionPath,
-            ComputedStyle::IncludeIndependentTransformProperties);
-        transform.scale(zoom);
-      } else {
-        style->applyTransform(
-            transform, layoutObject()->objectBoundingBox(),
-            ComputedStyle::IncludeTransformOrigin,
-            ComputedStyle::IncludeMotionPath,
-            ComputedStyle::IncludeIndependentTransformProperties);
-      }
-    }
-
-    // Flatten any 3D transform.
-    matrix = transform.toAffineTransform();
-  } else {
-    m_transform->currentValue()->concatenate(matrix);
-  }
-
+  AffineTransform matrix = calculateTransform();
   if (hasSVGRareData())
-    return *svgRareData()->animateMotionTransform() * matrix;
+    matrix.preMultiply(*svgRareData()->animateMotionTransform());
   return matrix;
 }
 
 AffineTransform* SVGGraphicsElement::animateMotionTransform() {
   return ensureSVGRareData()->animateMotionTransform();
 }
 
 void SVGGraphicsElement::svgAttributeChanged(const QualifiedName& attrName) {
   // Reattach so the isValid() check will be run again during layoutObject
   // creation.
   if (SVGTests::isKnownAttribute(attrName)) {
     SVGElement::InvalidationGuard invalidationGuard(this);
     lazyReattachIfAttached();
     return;
   }
 
   if (attrName == SVGNames::transformAttr) {
     LayoutObject* object = layoutObject();
     if (!object)
       return;
 
+    invalidateSVGPresentationAttributeStyle();
+
     SVGElement::InvalidationGuard invalidationGuard(this);
-    object->setNeedsTransformUpdate();
+    // TODO(fs): The InvalidationGuard will make sure all instances are
+    // invalidated, but the style recalc will propagate to instances too. So
+    // there is some redundant operations being performed here. Could we get
+    // away with removing the InvalidationGuard?
+    setNeedsStyleRecalc(LocalStyleChange,
+                        StyleChangeReasonForTracing::fromAttribute(attrName));
     markForLayoutAndParentResourceInvalidation(object);
     return;
   }
 
   SVGElement::svgAttributeChanged(attrName);
 }
 
 SVGElement* SVGGraphicsElement::nearestViewportElement() const {
   for (Element* current = parentOrShadowHostElement(); current;
        current = current->parentOrShadowHostElement()) {
@@ skipping 23 matching lines @@
 
   return layoutObject()->objectBoundingBox();
 }
 
 SVGRectTearOff* SVGGraphicsElement::getBBoxFromJavascript() {
   return SVGRectTearOff::create(SVGRect::create(getBBox()), 0,
                                 PropertyIsNotAnimVal);
 }
 
 }  // namespace blink