Change SimpleFontData::platformBoundsForGlyph to use getTextPath

Change SimpleFontData::platformBoundsForGlyph to use getTextPath

On skia platforms we currnetly implement platformBoundsForGlyph by
using the measureText API. This returns a conservative bounding box
intended for paint but it is close enough to the actual metrics in
most cases. For vertical text however it tends to overly inflated.

Change it to use the getTextPath API instead which returns more
precise metrics instead for text metrics.

BUG=249099
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=158584

[eae, 2013-09-30 22:43:26]

The new pixel results all show _less_ red than before and for the text ones
certain glyphs shift by one pixel.

[leviw_travelin_and_unemployed, 2013-09-30 22:46:59]

nit: "concervative" in description.

LGTM.

https://codereview.chromium.org/25045010/diff/6001/Source/core/platform/graph...
File Source/core/platform/graphics/skia/SimpleFontDataSkia.cpp (right):

https://codereview.chromium.org/25045010/diff/6001/Source/core/platform/graph...
Source/core/platform/graphics/skia/SimpleFontDataSkia.cpp:227: // fallback can
go away.
Is there a crbug for this? If not please file.

[eae, 2013-09-30 22:48:57]

On 2013/09/30 22:46:59, Levi wrote:
> nit: "concervative" in description.

Fixed, thanks.

>
https://codereview.chromium.org/25045010/diff/6001/Source/core/platform/graph...
> Source/core/platform/graphics/skia/SimpleFontDataSkia.cpp:227: // fallback can
> go away.
> Is there a crbug for this? If not please file.

It's already been fixed in Skia, I'll remove this after the next Skia roll.

[commit-bot: I haz the power, 2013-09-30 22:49:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/eae@chromium.org/25045010/6001

[commit-bot: I haz the power, 2013-10-01 03:13:06]

Change committed as 158584