diff options
| author |  mtklein <mtklein@chromium.org> | 2015-09-15 13:29:20 -0700 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2015-09-15 13:29:20 -0700 |
| commit | ca6f43b6fd2c650a60e38f5080b42c9448c21dea (patch) | |
| tree | ead1fadcc606296261c71be2748ea96212da6fa6 /tools/VisualBench | |
| parent | 27b6335c71bccb96206ff0495793433bfa982777 (diff) | |
vb: also thread --key through to .json
BUG=skia:
Review URL: https://codereview.chromium.org/1342203002
Diffstat (limited to 'tools/VisualBench')
| -rw-r--r-- | tools/VisualBench/VisualLightweightBenchModule.cpp | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/tools/VisualBench/VisualLightweightBenchModule.cpp b/tools/VisualBench/VisualLightweightBenchModule.cpp index bc592a3934..7b7cdbbeb7 100644 --- a/tools/VisualBench/VisualLightweightBenchModule.cpp +++ b/tools/VisualBench/VisualLightweightBenchModule.cpp @@ -31,8 +31,9 @@ DEFINE_int32(samples, 10, "Number of times to time each skp."); DEFINE_int32(frames, 5, "Number of frames of each skp to render per sample."); DEFINE_double(loopMs, 5, "Target loop time in millseconds."); DEFINE_bool2(verbose, v, false, "enable verbose output from the test driver."); -DEFINE_string(key, "", ""); // dummy to enable gm tests that have platform-specific names DEFINE_string(outResultsFile, "", "If given, write results here as JSON."); +DEFINE_string(key, "", + "Space-separated key/value pairs to add to JSON identifying this builder."); DEFINE_string(properties, "", "Space-separated key/value pairs to add to JSON identifying this run."); @@ -63,6 +64,14 @@ VisualLightweightBenchModule::VisualLightweightBenchModule(VisualBench* owner) fResults.reset(new NanoJSONResultsWriter(FLAGS_outResultsFile[0])); } + if (1 == FLAGS_key.count() % 2) { + SkDebugf("ERROR: --key must be passed with an even number of arguments.\n"); + } else { + for (int i = 1; i < FLAGS_key.count(); i += 2) { + fResults->key(FLAGS_key[i - 1], FLAGS_key[i]); + } + } + if (1 == FLAGS_properties.count() % 2) { SkDebugf("ERROR: --properties must be passed with an even number of arguments.\n"); } else { |