Remove isSupportedAttribute in svg

Source/core/svg/SVGTextContentElement.cpp

 /*
  * Copyright (C) 2004, 2005, 2007, 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 182 matching lines @@
         start = start.next();
 
     // Find selection end
     VisiblePosition end(start);
     for (unsigned i = 0; i < nchars; ++i)
         end = end.next();
 
     document().frame()->selection().setSelection(VisibleSelection(start, end));
 }
 
-bool SVGTextContentElement::isSupportedAttribute(const QualifiedName& attrName)
-{
-    DEFINE_STATIC_LOCAL(HashSet<QualifiedName>, supportedAttributes, ());
-    if (supportedAttributes.isEmpty()) {
-        supportedAttributes.add(SVGNames::lengthAdjustAttr);
-        supportedAttributes.add(SVGNames::textLengthAttr);
-        supportedAttributes.add(XMLNames::spaceAttr);
-    }
-    return supportedAttributes.contains<SVGAttributeHashTranslator>(attrName);
-}
-
 bool SVGTextContentElement::isPresentationAttribute(const QualifiedName& name) const
 {
     if (name.matches(XMLNames::spaceAttr))
         return true;
     return SVGGraphicsElement::isPresentationAttribute(name);
 }
 
 void SVGTextContentElement::collectStyleForPresentationAttribute(const QualifiedName& name, const AtomicString& value, MutableStylePropertySet* style)
 {
-    if (!isSupportedAttribute(name))
-        SVGGraphicsElement::collectStyleForPresentationAttribute(name, value, style);
-    else if (name.matches(XMLNames::spaceAttr)) {
+    if (name.matches(XMLNames::spaceAttr)) {
         DEFINE_STATIC_LOCAL(const AtomicString, preserveString, ("preserve", AtomicString::ConstructFromLiteral));
 
         if (value == preserveString) {
             UseCounter::count(document(), UseCounter::WhiteSpacePreFromXMLSpace);
             addPropertyToPresentationAttributeStyle(style, CSSPropertyWhiteSpace, CSSValuePre);
         } else {
             UseCounter::count(document(), UseCounter::WhiteSpaceNowrapFromXMLSpace);
             addPropertyToPresentationAttributeStyle(style, CSSPropertyWhiteSpace, CSSValueNowrap);
         }
+    } else {
+        SVGGraphicsElement::collectStyleForPresentationAttribute(name, value, style);
     }
 }
 
 void SVGTextContentElement::svgAttributeChanged(const QualifiedName& attrName)
 {
-    if (!isSupportedAttribute(attrName)) {
-        SVGGraphicsElement::svgAttributeChanged(attrName);
+    if (attrName == SVGNames::textLengthAttr)
+        m_textLengthIsSpecifiedByUser = true;
+
+    if (attrName == SVGNames::textLengthAttr
+        || attrName == SVGNames::lengthAdjustAttr
+        || attrName == XMLNames::spaceAttr) {
+        SVGElement::InvalidationGuard invalidationGuard(this);
+
+        if (LayoutObject* renderer = this->layoutObject())
+            markForLayoutAndParentResourceInvalidation(renderer);
+
         return;
     }
 
-    if (attrName == SVGNames::textLengthAttr)
-        m_textLengthIsSpecifiedByUser = true;
-
-    SVGElement::InvalidationGuard invalidationGuard(this);
-
-    if (LayoutObject* renderer = this->layoutObject())
-        markForLayoutAndParentResourceInvalidation(renderer);
+    SVGGraphicsElement::svgAttributeChanged(attrName);
 }
 
 bool SVGTextContentElement::selfHasRelativeLengths() const
 {
     // Any element of the <text> subtree is advertized as using relative lengths.
     // On any window size change, we have to relayout the text subtree, as the
     // effective 'on-screen' font size may change.
     return true;
 }
 
 SVGTextContentElement* SVGTextContentElement::elementFromRenderer(LayoutObject* renderer)
 {
     if (!renderer)
         return 0;
 
     if (!renderer->isSVGText() && !renderer->isSVGInline())
         return 0;
 
     SVGElement* element = toSVGElement(renderer->node());
     ASSERT(element);
     return isSVGTextContentElement(*element) ? toSVGTextContentElement(element) : 0;
 }
 
 }