Merge 126657

Source/WebCore/svg/SVGTests.cpp

 /*
  * Copyright (C) 2004, 2005, 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 137 matching lines @@
         || 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->attached())
+    bool attached = targetElement->attached();
+    if (valid && !attached && targetElement->parentNode()->attached())
         targetElement->attach();
-    if (!valid && targetElement->attached())
+    else if (!valid && attached)
         targetElement->detach();
+
     return true;
 }
 
 void SVGTests::addSupportedAttributes(HashSet<QualifiedName>& supportedAttributes)
 {
     supportedAttributes.add(SVGNames::requiredFeaturesAttr);
     supportedAttributes.add(SVGNames::requiredExtensionsAttr);
     supportedAttributes.add(SVGNames::systemLanguageAttr);
 }
 
@@ skipping 38 matching lines @@
 
 SVGStringList& SVGTests::systemLanguage()
 {
     m_systemLanguage.shouldSynchronize = true;
     return m_systemLanguage.value;
 }
 
 }
 
 #endif // ENABLE(SVG)