Use distance fields for glyphs > 256 pt, before switching to paths.

Use distance fields for glyphs > 256 pt, before switching to paths.

BUG=452313
Committed: https://skia.googlesource.com/skia/+/5a3d92fca17b9d4d7763c6d24c72c74f7682881f

[jvanverth1, 2015-01-27 20:31:23]

jvanverth@google.com changed reviewers:
+ robertphillips@google.com

[robertphillips, 2015-01-28 00:03:16]

lgtm + a couple of questions

https://codereview.chromium.org/862403004/diff/1/src/gpu/GrDistanceFieldTextC...
File src/gpu/GrDistanceFieldTextContext.cpp (left):

https://codereview.chromium.org/862403004/diff/1/src/gpu/GrDistanceFieldTextC...
src/gpu/GrDistanceFieldTextContext.cpp:100: //       enable perspective
Is ShouldDrawTextAsPaths still used anywhere?

https://codereview.chromium.org/862403004/diff/1/src/gpu/GrDistanceFieldTextC...
File src/gpu/GrDistanceFieldTextContext.cpp (right):

https://codereview.chromium.org/862403004/diff/1/src/gpu/GrDistanceFieldTextC...
src/gpu/GrDistanceFieldTextContext.cpp:91: SkScalar scaledTextSize =
maxScale*paint.getTextSize();
// Scaling up beyond 2x yields undesireable artifacts ?

[jvanverth1, 2015-01-28 17:47:39]

https://codereview.chromium.org/862403004/diff/1/src/gpu/GrDistanceFieldTextC...
File src/gpu/GrDistanceFieldTextContext.cpp (left):

https://codereview.chromium.org/862403004/diff/1/src/gpu/GrDistanceFieldTextC...
src/gpu/GrDistanceFieldTextContext.cpp:100: //       enable perspective
On 2015/01/28 00:03:16, robertphillips wrote:
> Is ShouldDrawTextAsPaths still used anywhere?

It's still used in BitmapTextContext, and in raster.

https://codereview.chromium.org/862403004/diff/1/src/gpu/GrDistanceFieldTextC...
File src/gpu/GrDistanceFieldTextContext.cpp (right):

https://codereview.chromium.org/862403004/diff/1/src/gpu/GrDistanceFieldTextC...
src/gpu/GrDistanceFieldTextContext.cpp:91: SkScalar scaledTextSize =
maxScale*paint.getTextSize();
On 2015/01/28 00:03:16, robertphillips wrote:
> // Scaling up beyond 2x yields undesireable artifacts ?

Done.

[jvanverth1, 2015-01-28 21:03:11]

The CQ bit was checked by jvanverth@google.com

[commit-bot: I haz the power, 2015-01-28 21:03:28]

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

[commit-bot: I haz the power, 2015-01-28 21:08:44]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/5a3d92fca17b9d4d7763c6d24c72c74f7682881f