Remove isSupportedAttribute in svg

Source/core/svg/SVGLineElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2008 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007 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 32 matching lines @@
 {
     visitor->trace(m_x1);
     visitor->trace(m_y1);
     visitor->trace(m_x2);
     visitor->trace(m_y2);
     SVGGeometryElement::trace(visitor);
 }
 
 DEFINE_NODE_FACTORY(SVGLineElement)
 
-bool SVGLineElement::isSupportedAttribute(const QualifiedName& attrName)
-{
-    DEFINE_STATIC_LOCAL(HashSet<QualifiedName>, supportedAttributes, ());
-    if (supportedAttributes.isEmpty()) {
-        supportedAttributes.add(SVGNames::x1Attr);
-        supportedAttributes.add(SVGNames::x2Attr);
-        supportedAttributes.add(SVGNames::y1Attr);
-        supportedAttributes.add(SVGNames::y2Attr);
-    }
-    return supportedAttributes.contains<SVGAttributeHashTranslator>(attrName);
-}
-
 void SVGLineElement::svgAttributeChanged(const QualifiedName& attrName)
 {
-    if (!isSupportedAttribute(attrName)) {
-        SVGGeometryElement::svgAttributeChanged(attrName);
-        return;
-    }
-
-    SVGElement::InvalidationGuard invalidationGuard(this);
-
     bool isLengthAttribute = attrName == SVGNames::x1Attr
                           || attrName == SVGNames::y1Attr
                           || attrName == SVGNames::x2Attr
                           || attrName == SVGNames::y2Attr;
 
     if (isLengthAttribute)
         updateRelativeLengthsInformation();
 
-    LayoutSVGShape* renderer = toLayoutSVGShape(this->layoutObject());
-    if (!renderer)
-        return;
+    if (isLengthAttribute) {

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

Merge blocks.

+        SVGElement::InvalidationGuard invalidationGuard(this);
 
-    if (isLengthAttribute) {
-        renderer->setNeedsShapeUpdate();
-        markForLayoutAndParentResourceInvalidation(renderer);
+        LayoutSVGShape* object = toLayoutSVGShape(this->layoutObject());
+        if (!object)
+            return;
+
+        object->setNeedsShapeUpdate();
+        markForLayoutAndParentResourceInvalidation(object);
         return;
     }
 
-    ASSERT_NOT_REACHED();
+    SVGGeometryElement::svgAttributeChanged(attrName);
 }
 
 bool SVGLineElement::selfHasRelativeLengths() const
 {
     return m_x1->currentValue()->isRelative()
         || m_y1->currentValue()->isRelative()
         || m_x2->currentValue()->isRelative()
         || m_y2->currentValue()->isRelative();
 }
 
 } // namespace blink