Don't try to add large glyphs to the font atlas.

Don't try to add large glyphs to the font atlas.

When the glyph data is invalid we can try to allocate a ridiculous amount of memory. This adds a check to cover such cases.

BUG=414581
Committed: https://skia.googlesource.com/skia/+/681e65b19940e10f39bf1237b4b35aa94677b3d0

[jvanverth1, 2014-09-19 19:39:55]

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

[egdaniel, 2014-09-19 19:49:30]

lgtm + possible comment

https://codereview.chromium.org/585853003/diff/1/src/gpu/GrTextStrike.h
File src/gpu/GrTextStrike.h (right):

https://codereview.chromium.org/585853003/diff/1/src/gpu/GrTextStrike.h#newco...
src/gpu/GrTextStrike.h:40: bool glyphTooLargeForAtlas(GrGlyph*);
Maybe add comment on what it means to be too large?

[jvanverth1, 2014-09-19 19:59:51]

The CQ bit was checked by jvanverth@google.com

[commit-bot: I haz the power, 2014-09-19 20:00:15]

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

[commit-bot: I haz the power, 2014-09-19 20:07:43]

Committed patchset #2 (id:20001) as 681e65b19940e10f39bf1237b4b35aa94677b3d0