Devirtualize supportsSpatialNavigationFocus

Source/core/dom/Element.cpp

Index: Source/core/dom/Element.cpp
diff --git a/Source/core/dom/Element.cpp b/Source/core/dom/Element.cpp
index 7b03118ae897022f701741706f20b8edaf64215f..de78070f2a3bc14f7f4aace169de7bed07d6c2b2 100644
--- a/Source/core/dom/Element.cpp
+++ b/Source/core/dom/Element.cpp
@@ -2182,7 +2182,12 @@ bool Element::supportsSpatialNavigationFocus() const
     return hasEventListeners(EventTypeNames::click)
         || hasEventListeners(EventTypeNames::keydown)
         || hasEventListeners(EventTypeNames::keypress)
-        || hasEventListeners(EventTypeNames::keyup);
+        || hasEventListeners(EventTypeNames::keyup)
+        || (isSVGElement() && (hasEventListeners(EventTypeNames::focus)

[pdr., 2014/07/10 21:19:23]

I don't think the indentation is correct here, or it's at least hard to read.
Can you refactor this so it's easy to tell that focus/blur/etc only apply to the
isSVGElement() case?


SVGElement doesn't seem to be special here. Should blur/focus/etc apply to all
elements and not just SVGElement?

+        || hasEventListeners(EventTypeNames::blur)
+        || hasEventListeners(EventTypeNames::focusin)
+        || hasEventListeners(EventTypeNames::focusout)));
+
 }
 
 bool Element::isFocusable() const