REGRESSION: Click on focused link should not remove focus on it.

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 085e5f637ef6c6d1cbf22c232a34f8ee7c72a3ab..8b418cb9e6c1c353606098feee3b971594c778d6 100644
--- a/Source/core/svg/SVGElement.cpp
+++ b/Source/core/svg/SVGElement.cpp
@@ -640,6 +640,17 @@ RenderStyle* SVGElement::computedStyle(PseudoId pseudoElementSpecifier)
     return svgRareData()->overrideComputedStyle(this, parentStyle);
 }
 
+bool SVGElement::hasFocusEventListeners() const
+{
+    SVGElement* mutableThis = const_cast<SVGElement*>(this);

[pdr., 2013/07/29 00:17:47]

I've been bitten by cost_cast in the past. If this is truly safe, can
hasEventListeners be made const and this const_cast moved there?

[tkent, 2013/07/29 03:31:07]

yeah, I think we can.  I added a FIXME comment.

+    return mutableThis->hasEventListeners(eventNames().focusinEvent) || mutableThis->hasEventListeners(eventNames().focusoutEvent);
+}
+
+bool SVGElement::isKeyboardFocusable(KeyboardEvent*) const
+{
+    return isFocusable();
+}
+
 #ifndef NDEBUG
 bool SVGElement::isAnimatableAttribute(const QualifiedName& name) const
 {