SVGTests should lazyAttach

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 e9c66316839f32a29fd7fcb53ca679e3eceb6505..871475f7e590a3bb312f4ae0edc8ec8123b03228 100644
--- a/Source/core/svg/SVGTests.cpp
+++ b/Source/core/svg/SVGTests.cpp
@@ -157,10 +157,9 @@ bool SVGTests::handleAttributeChange(SVGElement* targetElement, const QualifiedN
         return true;
 
     bool valid = targetElement->isValid();
-    bool attached = targetElement->attached();
-    if (valid && !attached && targetElement->parentNode()->attached())
-        targetElement->attach();
-    else if (!valid && attached)
+    if (valid && !targetElement->attached())

[pdr., 2013/05/18 03:37:59]

You removed the parentNode check here which was added in
http://trac.webkit.org/changeset/126657 to fix a security bug. Can you help me
understand why that's safe?

+        targetElement->lazyAttach();
+    else if (!valid && targetElement->attached())
         targetElement->detach();
 
     return true;