add tests for SkDocument

add tests for SkDocument

Committed: http://code.google.com/p/skia/source/detail?r=11907

[reed1, 2013-10-21 18:54:28]

lgtm

[vandebo (ex-Chrome), 2013-10-21 19:02:42]

LGTM

https://codereview.chromium.org/33423002/diff/1/tests/DocumentTest.cpp
File tests/DocumentTest.cpp (right):

https://codereview.chromium.org/33423002/diff/1/tests/DocumentTest.cpp#newcode56
tests/DocumentTest.cpp:56: // If a file is created, it should be at least empty.
It may be better to define the behavior we desire and enforce it.

https://codereview.chromium.org/33423002/diff/1/tests/DocumentTest.cpp#newcod...
tests/DocumentTest.cpp:102: void test_PdfDpi(skiatest::Reporter* reporter) {
nit: omit until it has something in it.

[edisonn, 2013-10-21 19:14:55]

https://codereview.chromium.org/33423002/diff/1/tests/DocumentTest.cpp
File tests/DocumentTest.cpp (right):

https://codereview.chromium.org/33423002/diff/1/tests/DocumentTest.cpp#newcode56
tests/DocumentTest.cpp:56: // If a file is created, it should be at least empty.
On 2013/10/21 19:02:43, vandebo wrote:
> It may be better to define the behavior we desire and enforce it.

Done.

https://codereview.chromium.org/33423002/diff/1/tests/DocumentTest.cpp#newcod...
tests/DocumentTest.cpp:102: void test_PdfDpi(skiatest::Reporter* reporter) {
On 2013/10/21 19:02:43, vandebo wrote:
> nit: omit until it has something in it.

Done.

[vandebo (ex-Chrome), 2013-10-21 19:19:15]

LGTM

[commit-bot: I haz the power, 2013-10-22 14:09:46]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/edisonn@google.com/33423002/80001

[commit-bot: I haz the power, 2013-10-22 14:20:31]

Retried try job too often on Build-Mac10.7-Clang-x86-Release-Trybot for step(s)
BuildEverything, BuildTests
http://skiabot-master.pogerlabs.com:10117/buildstatus?builder=Build-Mac10.7-C...

[commit-bot: I haz the power, 2013-10-22 14:37:26]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/edisonn@google.com/33423002/200001

[commit-bot: I haz the power, 2013-10-22 14:49:05]

Change committed as 11907