Make anchors mouse-focusable

Source/core/dom/Node.cpp

Index: Source/core/dom/Node.cpp
diff --git a/Source/core/dom/Node.cpp b/Source/core/dom/Node.cpp
index 86b776ca6635bcd78088e9361e474924b4363aff..002b1f1e9bd760ceb93159253260d0de4dae199f 100644
--- a/Source/core/dom/Node.cpp
+++ b/Source/core/dom/Node.cpp
@@ -81,6 +81,7 @@
 #include "core/html/HTMLStyleElement.h"
 #include "core/page/ContextMenuController.h"
 #include "core/page/EventHandler.h"
+#include "core/page/FocusController.h"
 #include "core/page/Page.h"
 #include "core/frame/Settings.h"
 #include "core/rendering/FlowThreadController.h"
@@ -2181,6 +2182,12 @@ void Node::defaultEventHandler(Event* event)
     const AtomicString& eventType = event->type();
     if (eventType == EventTypeNames::keydown || eventType == EventTypeNames::keypress) {
         if (event->isKeyboardEvent()) {
+            if (eventType == EventTypeNames::keydown) {
+                if (Page* page = document().page()) {
+                    if (page->focusController().setWasFocusedByMouse(false) && renderer())
+                        renderer()->setShouldDoFullPaintInvalidation(true);
+                }
+            }

[robwu, 2014/08/19 17:03:56]

These lines are bugging me a little bit.
Previously, setShouldDoFullPaintInvalidation() was only called when the mouse
focus state changed for some nodes (form control elements and links). Now it is
called for *every* node.

That doesn't sound great in terms of performance, does it?
I'm thinking of moving back the focus logic to nodes via a template classes,
which also fixes the bug mentioned in c13. WDYT?

             if (LocalFrame* frame = document().frame())
                 frame->eventHandler().defaultKeyboardEventHandler(toKeyboardEvent(event));
         }
@@ -2237,10 +2244,6 @@ void Node::defaultEventHandler(Event* event)
     }
 }
 
-void Node::willCallDefaultEventHandler(const Event&)
-{
-}
-
 bool Node::willRespondToMouseMoveEvents()
 {
     if (isDisabledFormControl(this))