add a verbose flag to skdiff that shows the progress and status of

add a verbose flag to skdiff that shows the progress and status of
each comparison

BUG=

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

[humper, 2014-05-28 17:04:46]

ptal -- I hate always silent tools

[bungeman-skia, 2014-05-28 17:34:27]

Seems like a nice idea.

https://codereview.chromium.org/302443012/diff/20001/tools/skdiff_main.cpp
File tools/skdiff_main.cpp (right):

https://codereview.chromium.org/302443012/diff/20001/tools/skdiff_main.cpp#ne...
tools/skdiff_main.cpp:319: #define VERBOSE_STATUS(status,color,filename) if
(verbose) printf( "[ " color " %10s " ANSI_COLOR_RESET " ] %s\n", status,
filename->c_str())
Can we ifdef SK_OS_WIN or something here so we don't end up printing the escape
codes on Window's cmd.exe? SetConsoleTextAttribute() would work there, but
probably not worth the hassle.

https://codereview.chromium.org/302443012/diff/20001/tools/skdiff_main.cpp#ne...
tools/skdiff_main.cpp:420: 
remove extra line?

https://codereview.chromium.org/302443012/diff/20001/tools/skdiff_main.cpp#ne...
tools/skdiff_main.cpp:455: 
remove extra line?

[humper, 2014-05-28 17:55:31]

https://codereview.chromium.org/302443012/diff/20001/tools/skdiff_main.cpp
File tools/skdiff_main.cpp (right):

https://codereview.chromium.org/302443012/diff/20001/tools/skdiff_main.cpp#ne...
tools/skdiff_main.cpp:319: #define VERBOSE_STATUS(status,color,filename) if
(verbose) printf( "[ " color " %10s " ANSI_COLOR_RESET " ] %s\n", status,
filename->c_str())
On 2014/05/28 17:34:28, bungeman1 wrote:
> Can we ifdef SK_OS_WIN or something here so we don't end up printing the
escape
> codes on Window's cmd.exe? SetConsoleTextAttribute() would work there, but
> probably not worth the hassle.

Done.

https://codereview.chromium.org/302443012/diff/20001/tools/skdiff_main.cpp#ne...
tools/skdiff_main.cpp:420: 
On 2014/05/28 17:34:28, bungeman1 wrote:
> remove extra line?

Done.

https://codereview.chromium.org/302443012/diff/20001/tools/skdiff_main.cpp#ne...
tools/skdiff_main.cpp:455: 
On 2014/05/28 17:34:28, bungeman1 wrote:
> remove extra line?

Done.

[bungeman-skia, 2014-05-28 18:01:43]

lgtm

[humper, 2014-05-28 18:04:32]

The CQ bit was checked by humper@google.com

[commit-bot: I haz the power, 2014-05-28 18:04:49]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/humper@google.com/302443012/40001

[commit-bot: I haz the power, 2014-05-28 18:26:01]

Change committed as 14923