Avoiding losing too much precision when hit-testing SVG <text>

Avoiding losing too much precision when hit-testing SVG <text>

Using flooredIntPoint() will lose all fractional precision - which is
too much considering that the location is actually at least a
LayoutPoint.
Use the HitTestLocation(const FloatPoint&) constructor instead - which does flooredLayoutPoint() behind scenes but also retains the FloatPoint.
Do the same for LayoutSVGForeignObject, since it has similar requirements.

BUG=466617
Committed: https://crrev.com/a8807dbd794e4c18199c569b453ffd1943257642
Cr-Commit-Position: refs/heads/master@{#380144}

[fs, 2016-03-09 14:09:51]

Description was changed from

==========
Avoiding losing too much precision when hit-testing SVG <text>

Using flooredIntPoint() will lose all fractional precision - which is
too much considering that the location is actually a LayoutPoint.
Use roundedLayoutPoint() instead (similar to LayoutSVGForeignObject) to
retain as much fractional precision as possible (from the FloatPoint).

BUG=466617
==========

to

==========
Avoiding losing too much precision when hit-testing SVG <text>

Using flooredIntPoint() will lose all fractional precision - which is
too much considering that the location is actually at least a
LayoutPoint.
Use the HitTestLocation(const FloatPoint&) constructor instead - which does
flooredLayoutPoint() behind scenes but also retains the FloatPoint.
Do the same for LayoutSVGForeignObject, since it has similar requirements.

BUG=466617
==========

[fs, 2016-03-09 14:09:51]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[Stephen Chennney, 2016-03-09 15:17:46]

schenney@chromium.org changed reviewers:
+ schenney@chromium.org

[Stephen Chennney, 2016-03-09 15:17:46]

lgtm

[fs, 2016-03-09 16:19:59]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-03-09 16:20:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1775363002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1775363002/40001

[commit-bot: I haz the power, 2016-03-09 16:24:43]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-09 16:25:59]

Description was changed from

==========
Avoiding losing too much precision when hit-testing SVG <text>

Using flooredIntPoint() will lose all fractional precision - which is
too much considering that the location is actually at least a
LayoutPoint.
Use the HitTestLocation(const FloatPoint&) constructor instead - which does
flooredLayoutPoint() behind scenes but also retains the FloatPoint.
Do the same for LayoutSVGForeignObject, since it has similar requirements.

BUG=466617
==========

to

==========
Avoiding losing too much precision when hit-testing SVG <text>

Using flooredIntPoint() will lose all fractional precision - which is
too much considering that the location is actually at least a
LayoutPoint.
Use the HitTestLocation(const FloatPoint&) constructor instead - which does
flooredLayoutPoint() behind scenes but also retains the FloatPoint.
Do the same for LayoutSVGForeignObject, since it has similar requirements.

BUG=466617

Committed: https://crrev.com/a8807dbd794e4c18199c569b453ffd1943257642
Cr-Commit-Position: refs/heads/master@{#380144}
==========

[commit-bot: I haz the power, 2016-03-09 16:26:00]

Patchset 3 (id:??) landed as
https://crrev.com/a8807dbd794e4c18199c569b453ffd1943257642
Cr-Commit-Position: refs/heads/master@{#380144}