Image maps should work for images with alt content

Source/core/layout/HitTestResult.cpp

 /*
  * Copyright (C) 2006, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2012 Nokia Corporation and/or its subsidiary(-ies)
  *
  * 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
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 141 matching lines @@
         }
     }
 }
 
 void HitTestResult::setInnerNode(Node* n)
 {
     m_innerPossiblyPseudoNode = n;
     if (n && n->isPseudoElement())
         n = toPseudoElement(n)->findAssociatedNode();
     m_innerNode = n;
+

[pdr., 2015/04/24 21:48:23]

Instead of duplicating this logic here, what if we combined this with the
similar code in LayoutImage::nodeAtPoint into
HitTestResult::setToImageMapForImage (or similar). It looks like
setToShadowHostIfInUserAgentShadowRoot is already being called for this codepath
so you could either add a call to setToImageMapForImage after
setToShadowHostIfInUserAgentShadowRoot or create a combined function called
setToUserAgentShadowRootAndUseImageMapIfNeeded (or similar).

+    Element* shadowHost = n->shadowHost();
+    if (!shadowHost || !isHTMLImageElement(shadowHost))
+        return;
+    HTMLMapElement* map = toHTMLImageElement(shadowHost)->treeScope().getImageMap(toHTMLImageElement(shadowHost)->fastGetAttribute(usemapAttr));
+    if (!map)
+        return;
+    if (HTMLAreaElement* area = map->areaForPoint(localPoint(), toLayoutBox(layoutObject())->contentBoxRect().size()))
+        m_innerNode = area;
 }
 
 void HitTestResult::setURLElement(Element* n)
 {
     m_innerURLElement = n;
 }
 
 void HitTestResult::setScrollbar(Scrollbar* s)
 {
     m_scrollbar = s;
@@ skipping 306 matching lines @@
     else if (isHTMLMapElement(m_innerNode))
         imageMapImageElement = toHTMLMapElement(m_innerNode)->imageElement();
 
     if (!imageMapImageElement)
         return m_innerNode.get();
 
     return imageMapImageElement;
 }
 
 } // namespace blink