Remove isSupportedAttribute in svg

Source/core/svg/SVGForeignObjectElement.cpp

 /*
  * Copyright (C) 2006 Apple Inc. All rights reserved.
  * Copyright (C) 2008 Nikolas Zimmermann <zimmermann@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 37 matching lines @@
 {
     visitor->trace(m_x);
     visitor->trace(m_y);
     visitor->trace(m_width);
     visitor->trace(m_height);
     SVGGraphicsElement::trace(visitor);
 }
 
 DEFINE_NODE_FACTORY(SVGForeignObjectElement)
 
-bool SVGForeignObjectElement::isSupportedAttribute(const QualifiedName& attrName)
-{
-    DEFINE_STATIC_LOCAL(HashSet<QualifiedName>, supportedAttributes, ());
-    if (supportedAttributes.isEmpty()) {
-        supportedAttributes.add(SVGNames::xAttr);
-        supportedAttributes.add(SVGNames::yAttr);
-        supportedAttributes.add(SVGNames::widthAttr);
-        supportedAttributes.add(SVGNames::heightAttr);
-    }
-    return supportedAttributes.contains<SVGAttributeHashTranslator>(attrName);
-}
-
 bool SVGForeignObjectElement::isPresentationAttribute(const QualifiedName& name) const
 {
     if (name == SVGNames::xAttr || name == SVGNames::yAttr
         || name == SVGNames::widthAttr || name == SVGNames::heightAttr)
         return true;
     return SVGGraphicsElement::isPresentationAttribute(name);
 }
 
 bool SVGForeignObjectElement::isPresentationAttributeWithSVGDOM(const QualifiedName& attrName) const
 {
@@ skipping 13 matching lines @@
     else if (property == m_x)
         addSVGLengthPropertyToPresentationAttributeStyle(style, CSSPropertyX, *m_x->currentValue());
     else if (property == m_y)
         addSVGLengthPropertyToPresentationAttributeStyle(style, CSSPropertyY, *m_y->currentValue());
     else
         SVGGraphicsElement::collectStyleForPresentationAttribute(name, value, style);
 }
 
 void SVGForeignObjectElement::svgAttributeChanged(const QualifiedName& attrName)
 {
-    if (!isSupportedAttribute(attrName)) {
-        SVGGraphicsElement::svgAttributeChanged(attrName);
-        return;
-    }
-
-    SVGElement::InvalidationGuard invalidationGuard(this);
-
     bool isWidthHeightAttribute = attrName == SVGNames::widthAttr
         || attrName == SVGNames::heightAttr;
     bool isXYAttribute = attrName == SVGNames::xAttr || attrName == SVGNames::yAttr;
 
     if (isXYAttribute || isWidthHeightAttribute) {
+        SVGElement::InvalidationGuard invalidationGuard(this);
+
         invalidateSVGPresentationAttributeStyle();
         setNeedsStyleRecalc(LocalStyleChange,
             isWidthHeightAttribute ? StyleChangeReasonForTracing::create(StyleChangeReason::SVGContainerSizeChange) : StyleChangeReasonForTracing::fromAttribute(attrName));
 
         updateRelativeLengthsInformation();
         if (LayoutObject* renderer = this->layoutObject())
             markForLayoutAndParentResourceInvalidation(renderer);
+
+        return;
     }
+
+    SVGGraphicsElement::svgAttributeChanged(attrName);
 }
 
 LayoutObject* SVGForeignObjectElement::createLayoutObject(const ComputedStyle&)
 {
     return new LayoutSVGForeignObject(this);
 }
 
 bool SVGForeignObjectElement::layoutObjectIsNeeded(const ComputedStyle& style)
 {
     // Suppress foreignObject renderers in SVG hidden containers.
@@ skipping 14 matching lines @@
 
 bool SVGForeignObjectElement::selfHasRelativeLengths() const
 {
     return m_x->currentValue()->isRelative()
         || m_y->currentValue()->isRelative()
         || m_width->currentValue()->isRelative()
         || m_height->currentValue()->isRelative();
 }
 
 } // namespace blink