Add infrastructure for testing the debugger code.

Add infrastructure for testing the debugger code.

This creates the necessarily files and GYP targets to start adding tests
for the debugger code. There is currently one test added, more will be added
once the infrastructure is in place.

BUG=

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

[dsinclair, 2013-09-11 15:06:38]

mtklein, PTAL. This is extracting the testing code from the other toString CL.

[mtklein, 2013-09-11 15:15:10]

On 2013/09/11 15:06:38, dsinclair wrote:
> mtklein, PTAL. This is extracting the testing code from the other toString CL.

lgtm

I have a suggestion here that I don't feel strongly about at all.  Do it or
don't.

Our continuous build is a little crazy just now, but I think ticking commit will
work, eventually.  If it doesn't I'll commit this for you once things settle.

[mtklein, 2013-09-11 15:15:28]

Ooops, meant to actually send the suggestion.

https://codereview.chromium.org/23589007/diff/1/tests/DebugCanvasTest.cpp
File tests/DebugCanvasTest.cpp (right):

https://codereview.chromium.org/23589007/diff/1/tests/DebugCanvasTest.cpp#new...
tests/DebugCanvasTest.cpp:18: const SkTDArray<SkString*>* result =
cmds[0]->Info();
This is just a suggestion.  We're assuming result != NULL here, which is just
fine by me, but let's write it as

const SkTDArray<SkString*>& result = *cmds[0]->Info();

with whatever parens are needed to make that work.  Then the code later on will
look a little less starry.

[dsinclair, 2013-09-11 15:20:44]

https://codereview.chromium.org/23589007/diff/1/tests/DebugCanvasTest.cpp
File tests/DebugCanvasTest.cpp (right):

https://codereview.chromium.org/23589007/diff/1/tests/DebugCanvasTest.cpp#new...
tests/DebugCanvasTest.cpp:18: const SkTDArray<SkString*>* result =
cmds[0]->Info();
On 2013/09/11 15:15:28, mtklein wrote:
> This is just a suggestion.  We're assuming result != NULL here, which is just
> fine by me, but let's write it as
> 
> const SkTDArray<SkString*>& result = *cmds[0]->Info();
> 
> with whatever parens are needed to make that work.  Then the code later on
will
> look a little less starry.

Done.

[commit-bot: I haz the power, 2013-09-11 15:21:04]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/dsinclair@chromium.org/23589007/6001

[commit-bot: I haz the power, 2013-09-11 15:28:49]

Retried try job too often on Build-Ubuntu12-GCC-x86_64-Release-Trybot for
step(s) BuildBench, BuildEverything, BuildGm, BuildSkiaLib, BuildTests,
BuildTools
http://173.255.115.253:10117/buildstatus?builder=Build-Ubuntu12-GCC-x86_64-Re...

[commit-bot: I haz the power, 2013-09-11 21:00:21]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/dsinclair@chromium.org/23589007/6001

[commit-bot: I haz the power, 2013-09-18 13:01:21]

Change committed as 11342

[robertphillips, 2013-09-18 13:29:52]

This was reverted in r11343 due to build breakages:

http://skiabot-master.pogerlabs.com:10117/builders/Build-Mac10.7-Clang-Arm7-D...

http://skiabot-master.pogerlabs.com:10117/builders/Build-Ubuntu12-GCC-Arm7-De...

http://skiabot-master.pogerlabs.com:10117/builders/Perf-Android-GalaxyNexus-S...