SVGTests should not leave detached elements in the tree

Source/core/svg/SVGTests.cpp

Index: Source/core/svg/SVGTests.cpp
diff --git a/Source/core/svg/SVGTests.cpp b/Source/core/svg/SVGTests.cpp
index e7d49ad5b45a238f4960a87d5711b7d5af97b310..9fc19277bdca9adf3f87ae4e59242cbe720d2a06 100644
--- a/Source/core/svg/SVGTests.cpp
+++ b/Source/core/svg/SVGTests.cpp
@@ -146,23 +146,6 @@ bool SVGTests::isKnownAttribute(const QualifiedName& attrName)
         || attrName == SVGNames::systemLanguageAttr;
 }
 
-bool SVGTests::handleAttributeChange(SVGElement* targetElement, const QualifiedName& attrName)
-{
-    ASSERT(targetElement);
-    if (!isKnownAttribute(attrName))
-        return false;
-    if (!targetElement->inDocument())
-        return true;
-
-    bool valid = targetElement->isValid();
-    if (valid && !targetElement->confusingAndOftenMisusedAttached() && targetElement->parentNode()->confusingAndOftenMisusedAttached())
-        targetElement->lazyAttach();
-    else if (!valid && targetElement->confusingAndOftenMisusedAttached())
-        targetElement->detach();
-
-    return true;
-}
-
 void SVGTests::addSupportedAttributes(HashSet<QualifiedName>& supportedAttributes)
 {
     supportedAttributes.add(SVGNames::requiredFeaturesAttr);