Remove isSupportedAttribute in svg

Source/core/svg/SVGTextPositioningElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Rob Buis <buis@kde.org>
  *
  * 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,
@@ skipping 36 matching lines @@
 DEFINE_TRACE(SVGTextPositioningElement)
 {
     visitor->trace(m_x);
     visitor->trace(m_y);
     visitor->trace(m_dx);
     visitor->trace(m_dy);
     visitor->trace(m_rotate);
     SVGTextContentElement::trace(visitor);
 }
 
-bool SVGTextPositioningElement::isSupportedAttribute(const QualifiedName& attrName)
-{
-    DEFINE_STATIC_LOCAL(HashSet<QualifiedName>, supportedAttributes, ());
-    if (supportedAttributes.isEmpty()) {
-        supportedAttributes.add(SVGNames::xAttr);
-        supportedAttributes.add(SVGNames::yAttr);
-        supportedAttributes.add(SVGNames::dxAttr);
-        supportedAttributes.add(SVGNames::dyAttr);
-        supportedAttributes.add(SVGNames::rotateAttr);
-    }
-    return supportedAttributes.contains<SVGAttributeHashTranslator>(attrName);
-}
-
 void SVGTextPositioningElement::svgAttributeChanged(const QualifiedName& attrName)
 {
-    if (!isSupportedAttribute(attrName)) {
-        SVGTextContentElement::svgAttributeChanged(attrName);
-        return;
-    }
-
-    SVGElement::InvalidationGuard invalidationGuard(this);
-
     bool updateRelativeLengths = attrName == SVGNames::xAttr
                               || attrName == SVGNames::yAttr
                               || attrName == SVGNames::dxAttr
                               || attrName == SVGNames::dyAttr;
 
     if (updateRelativeLengths)
         updateRelativeLengthsInformation();
 
-    LayoutObject* renderer = this->layoutObject();
-    if (!renderer)
+    if (updateRelativeLengths || attrName == SVGNames::rotateAttr) {
+        SVGElement::InvalidationGuard invalidationGuard(this);
+
+        LayoutObject* renderer = this->layoutObject();
+        if (!renderer)
+            return;
+
+        if (LayoutSVGText* textRenderer = LayoutSVGText::locateLayoutSVGTextAncestor(renderer))
+            textRenderer->setNeedsPositioningValuesUpdate();
+        markForLayoutAndParentResourceInvalidation(renderer);
+

[fs, 2015/04/10 14:25:24]

Drop.

[Erik Dahlström (inactive), 2015/04/10 15:33:44]

Done.

         return;
+    }
 
-    ASSERT(updateRelativeLengths || attrName == SVGNames::rotateAttr);
-
-    if (LayoutSVGText* textRenderer = LayoutSVGText::locateLayoutSVGTextAncestor(renderer))
-        textRenderer->setNeedsPositioningValuesUpdate();
-    markForLayoutAndParentResourceInvalidation(renderer);
+    SVGTextContentElement::svgAttributeChanged(attrName);
 }
 
 SVGTextPositioningElement* SVGTextPositioningElement::elementFromRenderer(LayoutObject& renderer)
 {
     if (!renderer.isSVGText() && !renderer.isSVGInline())
         return 0;
 
     Node* node = renderer.node();
     ASSERT(node);
     ASSERT(node->isSVGElement());
 
     return isSVGTextPositioningElement(*node) ? toSVGTextPositioningElement(node) : 0;
 }
 
 }