Add some text blob bounds unit tests.

Add some text blob bounds unit tests.

R=robertphillips@google.com

Committed: https://skia.googlesource.com/skia/+/51bf9573b8b875aec47cbeb9f4649f58bcbf56b7

[f(malita), 2014-08-22 13:58:30]

Includes some comments I missed in https://codereview.chromium.org/493443004/.

[robertphillips, 2014-08-22 14:23:49]

lgtm modulo some questions

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp
File tests/TextBlobTest.cpp (right):

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp#newco...
tests/TextBlobTest.cpp:141: SkAutoTUnref<const SkTextBlob>
blob(builder.build());
Why 65,65 rather then 50,50 ?

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp#newco...
tests/TextBlobTest.cpp:144: 
Isn't this the same as the first test ?

[f(malita), 2014-08-22 14:30:51]

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp
File tests/TextBlobTest.cpp (right):

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp#newco...
tests/TextBlobTest.cpp:141: SkAutoTUnref<const SkTextBlob>
blob(builder.build());
On 2014/08/22 14:23:48, robertphillips wrote:
> Why 65,65 rather then 50,50 ?

The union is [0 5 65 70] => the size is 65 x 65

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp#newco...
tests/TextBlobTest.cpp:144: 
On 2014/08/22 14:23:48, robertphillips wrote:
> Isn't this the same as the first test ?

It is, but this one verifies that the builder bbox is reset after being used to
build some non-empty blobs.

[robertphillips, 2014-08-22 14:32:50]

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp
File tests/TextBlobTest.cpp (right):

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp#newco...
tests/TextBlobTest.cpp:144: 
Can you add that as a comment?

[f(malita), 2014-08-22 14:44:18]

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp
File tests/TextBlobTest.cpp (right):

https://codereview.chromium.org/498693002/diff/1/tests/TextBlobTest.cpp#newco...
tests/TextBlobTest.cpp:144: 
On 2014/08/22 14:32:50, robertphillips wrote:
> Can you add that as a comment?

Done.

[f(malita), 2014-08-22 14:44:22]

The CQ bit was checked by fmalita@chromium.org

[commit-bot: I haz the power, 2014-08-22 14:44:34]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/fmalita@chromium.org/498693002/20001

[commit-bot: I haz the power, 2014-08-22 14:50:52]

Committed patchset #2 (20001) as 51bf9573b8b875aec47cbeb9f4649f58bcbf56b7