Use the correct bounding rect in SVGInlineTextBox::nodeAtPoint

third_party/WebKit/Source/core/layout/svg/line/SVGInlineTextBox.cpp

 /**
  * Copyright (C) 2007 Rob Buis <buis@kde.org>
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) Research In Motion Limited 2010. 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 226 matching lines @@
 {
     // FIXME: integrate with InlineTextBox::nodeAtPoint better.
     ASSERT(!isLineBreak());
 
     PointerEventsHitRules hitRules(PointerEventsHitRules::SVG_TEXT_HITTESTING, result.hitTestRequest(), getLineLayoutItem().style()->pointerEvents());
     bool isVisible = getLineLayoutItem().style()->visibility() == VISIBLE;
     if (isVisible || !hitRules.requireVisible) {
         if (hitRules.canHitBoundingBox
             || (hitRules.canHitStroke && (getLineLayoutItem().style()->svgStyle().hasStroke() || !hitRules.requireStroke))
             || (hitRules.canHitFill && (getLineLayoutItem().style()->svgStyle().hasFill() || !hitRules.requireFill))) {
-            LayoutPoint boxOrigin(x(), y());
-            boxOrigin.moveBy(accumulatedOffset);
-            LayoutRect rect(boxOrigin, size());
+            LayoutRect rect(topLeft(), logicalSize());

[Stephen Chennney, 2016/07/06 14:47:37]

You can do "LayoutRect rect(topLeft(), LayoutSize(logicalWidth(),
logicalheight()));", right?

[fs, 2016/07/06 15:00:51]

Yes. Done, and dropped method.

+            rect.moveBy(accumulatedOffset);
             if (locationInContainer.intersects(rect)) {
                 LineLayoutSVGInlineText lineLayoutItem = LineLayoutSVGInlineText(this->getLineLayoutItem());
                 ASSERT(lineLayoutItem.scalingFactor());
                 float baseline = lineLayoutItem.scaledFont().getFontMetrics().floatAscent() / lineLayoutItem.scalingFactor();
 
                 FloatPoint floatLocation = FloatPoint(locationInContainer.point());
                 for (const SVGTextFragment& fragment : m_textFragments) {
                     FloatQuad fragmentQuad = fragment.boundingQuad(baseline);
                     if (fragmentQuad.containsPoint(floatLocation)) {
                         lineLayoutItem.updateHitTestResult(result, locationInContainer.point() - toLayoutSize(accumulatedOffset));
                         if (result.addNodeToListBasedTestResult(lineLayoutItem.node(), locationInContainer, rect) == StopHitTesting)
                             return true;
                     }
                 }
             }
         }
     }
     return false;
 }
 
 } // namespace blink