VisualBenchTest: Write JSON output

VisualBenchTest: Write JSON output

BUG=skia:4093
Committed: https://skia.googlesource.com/skia/+/877a52ae61f41f0b519a906ed2460d44746de8e1

[borenet, 2015-08-20 14:05:15]

borenet@google.com changed reviewers:
+ djsollen@google.com, joshualitt@google.com, mtklein@google.com

[borenet, 2015-08-20 14:05:16]

https://codereview.chromium.org/1302103002/diff/1/bench/ResultsWriter.h
File bench/ResultsWriter.h (right):

https://codereview.chromium.org/1302103002/diff/1/bench/ResultsWriter.h#newco...
bench/ResultsWriter.h:125: }
I don't know that this is the right place for this.  It could also go in
VisualBench.cpp.  Also not sure that it's the right way to do it.  If I were
writing Python or Go, I'd just try to mkdir and if it failed I'd verify that the
directory already exists.

https://codereview.chromium.org/1302103002/diff/1/platform_tools/android/apps...
File
platform_tools/android/apps/visualbench/src/main/java/com/skia/VisualBenchTestActivity.java
(right):

https://codereview.chromium.org/1302103002/diff/1/platform_tools/android/apps...
platform_tools/android/apps/visualbench/src/main/java/com/skia/VisualBenchTestActivity.java:32:
intent.putExtra("cmdLineFlags", "--outResultsFile
/sdcard/skia_results/visualbench.json");
Eventually I need to figure out how to pass this value dynamically.  For now
it's hard-coded.

[djsollen, 2015-08-20 14:42:35]

https://codereview.chromium.org/1302103002/diff/1/platform_tools/android/apps...
File platform_tools/android/apps/robotium.cfg (right):

https://codereview.chromium.org/1302103002/diff/1/platform_tools/android/apps...
platform_tools/android/apps/robotium.cfg:1: [robotium]
what is this file for? I can guess, but having some kind of comment would be
helpful for future readers.

[borenet, 2015-08-20 15:32:23]

https://codereview.chromium.org/1302103002/diff/1/platform_tools/android/apps...
File platform_tools/android/apps/robotium.cfg (right):

https://codereview.chromium.org/1302103002/diff/1/platform_tools/android/apps...
platform_tools/android/apps/robotium.cfg:1: [robotium]
On 2015/08/20 14:42:35, djsollen wrote:
> what is this file for? I can guess, but having some kind of comment would be
> helpful for future readers.

Done.

[djsollen, 2015-08-20 15:57:42]

lgtm

[borenet, 2015-08-20 16:04:58]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2015-08-20 16:05:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1302103002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1302103002/20001

[commit-bot: I haz the power, 2015-08-20 16:12:42]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/877a52ae61f41f0b519a906ed2460d44746de8e1