Stop hit testing culled inlines when a match is found

third_party/WebKit/Source/core/layout/LayoutInline.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  *
  * 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.
  *
@@ skipping 783 matching lines @@
 };
 
 } // unnamed namespace
 
 bool LayoutInline::hitTestCulledInline(HitTestResult& result, const HitTestLocation& locationInContainer, const LayoutPoint& accumulatedOffset)
 {
     ASSERT(!alwaysCreateLineBoxes());
     if (!visibleToHitTestRequest(result.hitTestRequest()))
         return false;
 
-    HitTestLocation tmpLocation(locationInContainer, -toLayoutSize(accumulatedOffset));
+    HitTestLocation adjustedLocation(locationInContainer, -toLayoutSize(accumulatedOffset));
 
     Region regionResult;
-    HitTestCulledInlinesGeneratorContext context(regionResult, tmpLocation);
+    HitTestCulledInlinesGeneratorContext context(regionResult, adjustedLocation);
     generateCulledLineBoxRects(context, this);
 
     if (context.intersected()) {
-        updateHitTestResult(result, tmpLocation.point());
-        // We can not use addNodeToListBasedTestResult to determine if we fully enclose the hit-test area
-        // because it can only handle rectangular targets.
-        result.addNodeToListBasedTestResult(node(), locationInContainer);
-        // We check if using list-based hit-test to continue hit testing.
-        if (!result.hitTestRequest().penetratingList())
-            return regionResult.contains(tmpLocation.boundingBox());
+        updateHitTestResult(result, adjustedLocation.point());
+        if (result.addNodeToListBasedTestResult(node(), adjustedLocation, regionResult) == StopHitTesting)
+            return true;
     }
     return false;
 }
 
 PositionWithAffinity LayoutInline::positionForPoint(const LayoutPoint& point)
 {
     // FIXME: Does not deal with relative positioned inlines (should it?)
     LayoutBlock* cb = containingBlock();
     if (firstLineBoxIncludingCulling()) {
         // This inline actually has a line box.  We must have clicked in the border/padding of one of these boxes.  We
@@ skipping 605 matching lines @@
 
 void LayoutInline::invalidateDisplayItemClients(const LayoutBoxModelObject& paintInvalidationContainer, PaintInvalidationReason invalidationReason) const
 {
     LayoutBoxModelObject::invalidateDisplayItemClients(paintInvalidationContainer, invalidationReason);
 
     for (InlineFlowBox* box = firstLineBox(); box; box = box->nextLineBox())
         paintInvalidationContainer.invalidateDisplayItemClientOnBacking(*box, invalidationReason);
 }
 
 } // namespace blink