adding unit test for textblob cache / GrBatchAtlas

adding unit test for textblob cache / GrBatchAtlas

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/e49109f3b599afa1f9fe85d241aa68e842a7264e

[joshualitt, 2015-07-17 18:22:39]

joshualitt@google.com changed reviewers:
+ bsalomon@google.com, bungeman@google.com, joshualitt@google.com

[joshualitt, 2015-07-17 18:22:39]

ptal.  Any other cases you think we might be interested in?  Adding Ben for the
fontmgr stuff.

[bungeman-skia, 2015-07-17 19:07:57]

https://codereview.chromium.org/1233253003/diff/1/tests/TextBlobCacheTest.cpp
File tests/TextBlobCacheTest.cpp (right):

https://codereview.chromium.org/1233253003/diff/1/tests/TextBlobCacheTest.cpp...
tests/TextBlobCacheTest.cpp:86: SkString sname;
Doesn't look like you're using anything in this block (sname, fs)? Just remove?

[bsalomon, 2015-07-17 19:12:50]

maybe a comment somewhere indicating what this is testing. otherwise lgtm

[joshualitt, 2015-07-17 19:39:58]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-07-17 19:39:59]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/1233253003/#ps20001
(title: "comment")

[commit-bot: I haz the power, 2015-07-17 19:40:29]

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

[commit-bot: I haz the power, 2015-07-17 19:47:44]

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

[bungeman-skia, 2015-07-17 19:49:10]

https://codereview.chromium.org/1233253003/diff/20001/tests/TextBlobCacheTest...
File tests/TextBlobCacheTest.cpp (right):

https://codereview.chromium.org/1233253003/diff/20001/tests/TextBlobCacheTest...
tests/TextBlobCacheTest.cpp:84: SkString familyName;
Doesn't look like you're using the family name here either.

https://codereview.chromium.org/1233253003/diff/20001/tests/TextBlobCacheTest...
tests/TextBlobCacheTest.cpp:88: SkFontStyle fs;
Still not using this? I don't see why you want to call set->getStyle at all?