Fix link highlights on links that contain images.

Source/web/LinkHighlight.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 31 matching lines @@
 #include "core/rendering/style/ShadowData.h"
 #include "platform/graphics/Color.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebAnimationCurve.h"
 #include "public/platform/WebCompositorSupport.h"
 #include "public/platform/WebFloatAnimationCurve.h"
 #include "public/platform/WebFloatPoint.h"
 #include "public/platform/WebRect.h"
 #include "public/platform/WebSize.h"
 #include "wtf/CurrentTime.h"
+#include "wtf/Vector.h"
 
 using namespace WebCore;
 
 namespace blink {
 
 class WebViewImpl;
 
 PassOwnPtr<LinkHighlight> LinkHighlight::create(Node* node, WebViewImpl* owningWebViewImpl)
 {
     return adoptPtr(new LinkHighlight(node, owningWebViewImpl));
@@ skipping 115 matching lines @@
 static void addQuadToPath(const FloatQuad& quad, Path& path)
 {
     // FIXME: Make this create rounded quad-paths, just like the axis-aligned case.
     path.moveTo(quad.p1());
     path.addLineTo(quad.p2());
     path.addLineTo(quad.p3());
     path.addLineTo(quad.p4());
     path.closeSubpath();
 }
 
+void LinkHighlight::computeQuads(Node* node, Vector<FloatQuad>& outQuads) const
+{
+    if (!node || !node->renderer())
+        return;
+
+    RenderObject* renderer = node->renderer();
+
+    // For inline elements, absoluteQuads will return a line box based on the line-height
+    // and font metrics, which is technically incorrect as replaced elements like images
+    // should use their intristic height and expand the linebox  as needed. To get an
+    // appropriately sized highlight we descend into the children and have them add their
+    // boxes.
+    if (renderer->isRenderInline()) {
+        for (Node* child = node->firstChild(); child; child = child->nextSibling())
+            computeQuads(child, outQuads);
+    } else {
+        renderer->absoluteQuads(outQuads);
+    }
+
+}
+
 bool LinkHighlight::computeHighlightLayerPathAndPosition(RenderLayer* compositingLayer)
 {
     if (!m_node || !m_node->renderer() || !m_currentGraphicsLayer)
         return false;
 
     ASSERT(compositingLayer);
 
     // Get quads for node in absolute coordinates.
     Vector<FloatQuad> quads;
-    m_node->renderer()->absoluteQuads(quads);
+    computeQuads(m_node.get(), quads);
     ASSERT(quads.size());
 
     // Adjust for offset between target graphics layer and the node's renderer.
     FloatPoint positionAdjust = IntPoint(m_currentGraphicsLayer->offsetFromRenderer());
 
     Path newPath;
-    for (unsigned quadIndex = 0; quadIndex < quads.size(); ++quadIndex) {
+    for (size_t quadIndex = 0; quadIndex < quads.size(); ++quadIndex) {
         FloatQuad absoluteQuad = quads[quadIndex];
         absoluteQuad.move(-positionAdjust.x(), -positionAdjust.y());
 
         // Transform node quads in target absolute coords to local coordinates in the compositor layer.
         FloatQuad transformedQuad;
         convertTargetSpaceQuadToCompositedLayer(absoluteQuad, m_node->renderer(), compositingLayer->renderer(), transformedQuad);
 
         // FIXME: for now, we'll only use rounded paths if we have a single node quad. The reason for this is that
         // we may sometimes get a chain of adjacent boxes (e.g. for text nodes) which end up looking like sausage
         // links: these should ideally be merged into a single rect before creating the path, but that's
@@ skipping 119 matching lines @@
     // Make sure we update geometry on the next callback from WebViewImpl::layout().
     m_geometryNeedsUpdate = true;
 }
 
 WebLayer* LinkHighlight::layer()
 {
     return clipLayer();
 }
 
 } // namespace WeKit