Skip the TIMESTAMP file.

Skip the TIMESTAMP file.

Allows run_decoding_tests.py to pass the name of the folder, so
Android will not fail on account of the long adb command.

R=borenet@google.com

Committed: https://code.google.com/p/skia/source/detail?r=9817

[borenet, 2013-06-28 19:50:38]

LGTM to get the bots fixed, but I'd prefer a more robust solution...

https://codereview.chromium.org/18175008/diff/1/tools/skimage_main.cpp
File tools/skimage_main.cpp (right):

https://codereview.chromium.org/18175008/diff/1/tools/skimage_main.cpp#newcod...
tools/skimage_main.cpp:502: return !basename.startsWith("TIMESTAMP");
How much better is this than just looking for file extensions we expect to be
able to decode?  For the record, I think the timestamp file is going away with
checksums, but it might be replaced with a JSON file containing checksums.

[scroggo, 2013-06-28 20:00:10]

https://codereview.chromium.org/18175008/diff/1/tools/skimage_main.cpp
File tools/skimage_main.cpp (right):

https://codereview.chromium.org/18175008/diff/1/tools/skimage_main.cpp#newcod...
tools/skimage_main.cpp:502: return !basename.startsWith("TIMESTAMP");
On 2013/06/28 19:50:38, borenet wrote:
> How much better is this than just looking for file extensions we expect to be
> able to decode?  For the record, I think the timestamp file is going away with
> checksums, but it might be replaced with a JSON file containing checksums.

It is shorter and faster.

Changed in the latest patch to compare to known image extensions.

[borenet, 2013-06-28 20:02:02]

LGTM

https://codereview.chromium.org/18175008/diff/5001/tools/skimage_main.cpp
File tools/skimage_main.cpp (right):

https://codereview.chromium.org/18175008/diff/5001/tools/skimage_main.cpp#new...
tools/skimage_main.cpp:499: };
I agree that this is cumbersome, but I think it's better than listing the
timestamp file by name.

[scroggo, 2013-06-28 20:04:46]

Committed patchset #2 manually as r9817 (presubmit successful).