The link should be activated by enter key on focusing the child element inside the anchor(relanding)

Source/core/html/HTMLAnchorElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Simon Hausmann <hausmann@kde.org>
  * Copyright (C) 2003, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 129 matching lines @@
     int x = absolutePosition.x();
     int y = absolutePosition.y();
     url.append('?');
     url.appendNumber(x);
     url.append(',');
     url.appendNumber(y);
 }
 
 void HTMLAnchorElement::defaultEventHandler(Event* event)
 {
-    if (isLink()) {
-        if (focused() && isEnterKeyKeydownEvent(event) && isLiveLink()) {
+    if (isLiveLink()) {
+        ASSERT(event->target());
+        Node* target = event->target()->toNode();
+        ASSERT(target);
+        if ((focused() || target->focused()) && isEnterKeyKeypressEvent(event)) {

[robwu, 2014/10/13 09:19:48]

Don't forget to update similar code in
src/third_party/WebKit/Source/core/svg/SVGAElement.cpp.

[Miyoung Shin(g), 2014/10/13 11:05:01]

Sure, I remember it.
But I mentioned earlier, I hope to separate the patch with HTML and SVG because
I need to understand SVG first.
Can I do it?

[robwu, 2014/10/13 11:29:33]

There is some duplicate logic in the implementations of SVGAElement and
HTMLAnchorElement. It makes sense to update both implementations at once, since
it's usually a matter of copy-paste (plus a few changes, occasionally).

             event->setDefaultHandled();
             dispatchSimulatedClick(event);
             return;
         }
 
         if (isLinkClick(event) && isLiveLink()) {
             handleClick(event);
             return;
         }
     }
@@ skipping 188 matching lines @@
     } else {
         request.setRequestContext(blink::WebURLRequest::RequestContextHyperlink);
         FrameLoadRequest frameRequest(&document(), request, getAttribute(targetAttr));
         frameRequest.setTriggeringEvent(event);
         if (hasRel(RelationNoReferrer))
             frameRequest.setShouldSendReferrer(NeverSendReferrer);
         frame->loader().load(frameRequest);
     }
 }
 
+bool isEnterKeyKeypressEvent(Event* event)
+{
+    return event->type() == EventTypeNames::keypress && event->isKeyboardEvent() && toKeyboardEvent(event)->keyIdentifier() == "Enter";
+}
+
 bool isEnterKeyKeydownEvent(Event* event)

[robwu, 2014/10/13 09:19:48]

And remove this method declaration (and its defenition in the .h file), since it
is no longer used after this CL.

[Miyoung Shin(g), 2014/10/13 11:05:01]

Sure, I remember it as well.
After checking SVG is done, I will get rid of it.

 {
     return event->type() == EventTypeNames::keydown && event->isKeyboardEvent() && toKeyboardEvent(event)->keyIdentifier() == "Enter";
 }
 
 bool isLinkClick(Event* event)
 {
     return event->type() == EventTypeNames::click && (!event->isMouseEvent() || toMouseEvent(event)->button() != RightButton);
 }
 
 bool HTMLAnchorElement::willRespondToMouseClickEvents()
@@ skipping 14 matching lines @@
             Vector<String> argv;
             argv.append("a");
             argv.append(fastGetAttribute(hrefAttr));
             activityLogger->logEvent("blinkAddElement", argv.size(), argv.data());
         }
     }
     return HTMLElement::insertedInto(insertionPoint);
 }
 
 }