use commandlineflags for tests/

use commandlineflags for tests/

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

[reed1, 2013-04-11 15:53:48]

note: might want an option to specify that a given string flag expects exactly 1
value (like most of ours) to inform the user if they accidentally are giving us
an array.

[scroggo, 2013-04-11 16:28:01]

> note: might want an option to specify that a given string
> flag expects exactly 1 value (like most of ours) to inform
> the user if they accidentally are giving us an array.

Good thinking. I created https://code.google.com/p/skia/issues/detail?id=1224 to
track that.

https://codereview.chromium.org/14047009/diff/1/gyp/tests.gyp
File gyp/tests.gyp (right):

https://codereview.chromium.org/14047009/diff/1/gyp/tests.gyp#newcode18
gyp/tests.gyp:18: '../tools/flags',
You can remove this and the compilation of SkCommmandLineFlags.cpp by instead
adding

'flags.gyp:flags',

to the dependencies.

https://codereview.chromium.org/14047009/diff/1/tests/skia_test.cpp
File tests/skia_test.cpp (right):

https://codereview.chromium.org/14047009/diff/1/tests/skia_test.cpp#newcode8
tests/skia_test.cpp:8: #include "../tools/flags/SkCommandLineFlags.h"
You should be able to remove the full path if you depend on the project instead.

[reed1, 2013-04-11 16:31:21]

done

[scroggo, 2013-04-11 16:31:59]

On 2013/04/11 16:31:21, reed1 wrote:
> done

lgtm

[commit-bot: I haz the power, 2013-04-11 16:34:41]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/reed@google.com/14047009/4

[commit-bot: I haz the power, 2013-04-11 16:54:12]

Change committed as 8616