Stop using SVGElementInstance in SVGElement

Source/core/svg/SVGElement.cpp

Index: Source/core/svg/SVGElement.cpp
diff --git a/Source/core/svg/SVGElement.cpp b/Source/core/svg/SVGElement.cpp
index b0fc2fd950e54b850baa1e190a3d40c5d5364ff5..bbd6e02c64e1ee003736f5592863625de9796426 100644
--- a/Source/core/svg/SVGElement.cpp
+++ b/Source/core/svg/SVGElement.cpp
@@ -42,7 +42,6 @@
 #include "core/rendering/svg/RenderSVGResourceContainer.h"
 #include "core/svg/SVGCursorElement.h"
 #include "core/svg/SVGDocumentExtensions.h"
-#include "core/svg/SVGElementInstance.h"
 #include "core/svg/SVGElementRareData.h"
 #include "core/svg/SVGGraphicsElement.h"
 #include "core/svg/SVGSVGElement.h"
@@ -528,28 +527,25 @@ SVGDocumentExtensions& SVGElement::accessDocumentSVGExtensions()
     return document().accessSVGExtensions();
 }
 
-void SVGElement::mapInstanceToElement(SVGElementInstance* instance)
+void SVGElement::mapInstanceToElement(SVGElement* instance)
 {
     ASSERT(instance);
 
     HashSet<SVGElement*>& instances = ensureSVGRareData()->elementInstances();
-    ASSERT(!instances.contains(instance->shadowTreeElement()));
+    ASSERT(!instances.contains(instance));
 

[pdr., 2014/05/07 22:27:23]

I think the assertion here was useful (similarly in removeInstanceMapping).
Could we add it back?

Something like:
ASSERT(instance->inUseShadowTree());

-    instances.add(instance->shadowTreeElement());
+    instances.add(instance);
 }
 
-void SVGElement::removeInstanceMapping(SVGElementInstance* instance)
+void SVGElement::removeInstanceMapping(SVGElement* instance)
 {
     ASSERT(instance);
     ASSERT(hasSVGRareData());
 
-    if (!instance->shadowTreeElement())
-        return;
-
     HashSet<SVGElement*>& instances = svgRareData()->elementInstances();
-    ASSERT(instances.contains(instance->shadowTreeElement()));
+    ASSERT(instances.contains(instance));
 
-    instances.remove(instance->shadowTreeElement());
+    instances.remove(instance);
 }
 
 const HashSet<SVGElement*>& SVGElement::instancesForElement() const