Truncate RenderText widths before comparing against bounds.

Truncate RenderText widths before comparing against bounds.

Make RenderText truncate floating-point widths before
comparing them against integer display bounds when
determining whether text should be elided.

RenderTextHarfBuzz returns fractional widths when subpixel
positioning is enabled. When rendering omnibox results,
these widths typically exceed the integer bounds, resulting
in text getting inappropriately elided.

BUG=402347
Committed: https://crrev.com/ec562a798a1835e0383e83bae7dc71dffcfc5fb0
Cr-Commit-Position: refs/heads/master@{#298626}

[Daniel Erat, 2014-10-07 22:26:48]

derat@chromium.org changed reviewers:
+ ckocagil@chromium.org, msw@chromium.org

[Daniel Erat, 2014-10-07 22:26:48]

i left a longer comment on the bug.

i'd like to add a unit tests for this, but i'm not exactly sure how yet. on
linux, subpixel positioning isn't configurable (i could change that), and on
other platforms, there's no guarantee that we'll even use it at all.

[msw, 2014-10-07 22:42:58]

lgtm

[Daniel Erat, 2014-10-07 23:29:32]

The CQ bit was checked by derat@chromium.org

[commit-bot: I haz the power, 2014-10-07 23:31:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/631363002/1

[commit-bot: I haz the power, 2014-10-08 00:57:43]

Committed patchset #1 (id:1) as 4987c11cb9127a6a99bffafc1564766429488d13

[commit-bot: I haz the power, 2014-10-08 00:59:49]

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

[ckocagil, 2014-10-10 16:07:22]

lgtm, thanks for fixing this!