commit ca6f43b6fd2c650a60e38f5080b42c9448c21dea
author mtklein <mtklein@chromium.org> Tue Sep 15 13:29:20 2015 -0700
committer Commit bot <commit-bot@chromium.org> Tue Sep 15 13:29:20 2015 -0700
tree ead1fadcc606296261c71be2748ea96212da6fa6
parent 27b6335c71bccb96206ff0495793433bfa982777

vb: also thread --key through to .json

BUG=skia:

Review URL: https://codereview.chromium.org/1342203002

tools/VisualBench/VisualLightweightBenchModule.cpp

diff --git a/tools/VisualBench/VisualLightweightBenchModule.cpp b/tools/VisualBench/VisualLightweightBenchModule.cpp
index bc592a3..7b7cdbb 100644
--- a/tools/VisualBench/VisualLightweightBenchModule.cpp
+++ b/tools/VisualBench/VisualLightweightBenchModule.cpp
@@ -31,8 +31,9 @@
 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 @@
         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 {