Do not implement EventTarget on SVGElementInstance

Source/core/events/EventPath.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Neither the name of Google Inc. nor the names of its
@@ skipping 21 matching lines @@
 #include "SVGNames.h"
 #include "core/dom/FullscreenElementStack.h"
 #include "core/dom/Touch.h"
 #include "core/dom/TouchList.h"
 #include "core/dom/shadow/InsertionPoint.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/events/FocusEvent.h"
 #include "core/events/MouseEvent.h"
 #include "core/events/TouchEvent.h"
 #include "core/events/TouchEventContext.h"
-#include "core/svg/SVGElementInstance.h"
 #include "core/svg/SVGUseElement.h"
 
 namespace WebCore {
 
 // SVG1.1 specified that the <use> instance tree would expose the target
 // element for events. This has been deprecated and will be removed.
 // See: crbug.com/313438
 static bool usesDeprecatedSVGUseTreeEventRules(Node* node)
 {
     return node->isSVGElement() && toSVGElement(node)->inUseShadowTree();
 }
 
 EventTarget* EventPath::eventTargetRespectingTargetRules(Node* referenceNode)
 {
     ASSERT(referenceNode);
 
     if (referenceNode->isPseudoElement())
         return referenceNode->parentNode();
 
     if (!usesDeprecatedSVGUseTreeEventRules(referenceNode))

[pdr., 2014/06/02 14:09:06]

I thought we could remove all svg-specific logic here, though I may be missing
something.

         return referenceNode;
 
-    // Spec: The event handling for the non-exposed tree works as if the referenced element had been textually included
-    // as a deeply cloned child of the 'use' element, except that events are dispatched to the SVGElementInstance objects.
     Node& rootNode = referenceNode->treeScope().rootNode();
     Element* shadowHostElement = rootNode.isShadowRoot() ? toShadowRoot(rootNode).host() : 0;
-    // At this time, SVG nodes are not supported in non-<use> shadow trees.
-    if (!isSVGUseElement(shadowHostElement))
-        return referenceNode;
-    SVGUseElement& useElement = toSVGUseElement(*shadowHostElement);
-    if (SVGElementInstance* instance = useElement.instanceForShadowTreeElement(referenceNode))
-        return instance;
+    if (isSVGUseElement(shadowHostElement))
+        return shadowHostElement;
 
     return referenceNode;
 }
 
 static inline bool inTheSameScope(ShadowRoot* shadowRoot, EventTarget* target)
 {
     return target->toNode() && target->toNode()->treeScope().rootNode() == shadowRoot;
 }
 
 static inline EventDispatchBehavior determineDispatchBehavior(Event* event, ShadowRoot* shadowRoot, EventTarget* target)
@@ skipping 294 matching lines @@
 }
 #endif
 
 void EventPath::trace(Visitor* visitor)
 {
     visitor->trace(m_nodeEventContexts);
     visitor->trace(m_event);
 }
 
 } // namespace