Match works correctly on visualbench

tools/VisualBench/VisualBenchmarkStream.cpp

Index: tools/VisualBench/VisualBenchmarkStream.cpp
diff --git a/tools/VisualBench/VisualBenchmarkStream.cpp b/tools/VisualBench/VisualBenchmarkStream.cpp
index 841fb5d11bc8d457b43b9d694d13628951a54684..67a06bb501cbc9cbfb155254ef61dd3f55b7b681 100644
--- a/tools/VisualBench/VisualBenchmarkStream.cpp
+++ b/tools/VisualBench/VisualBenchmarkStream.cpp
@@ -65,6 +65,13 @@ bool VisualBenchmarkStream::ReadPicture(const char* path, SkAutoTUnref<SkPicture
 }
 
 Benchmark* VisualBenchmarkStream::next() {
+    Benchmark* bench;

[robertphillips, 2015/07/15 15:25:40]

// skip non-matching benchs
?

+    while ((bench = this->innerNext()) &&

[robertphillips, 2015/07/15 15:25:40]

Maybe put the '}' on its own line ?

+           SkCommandLineFlags::ShouldSkip(FLAGS_match, bench->getUniqueName())) {}
+    return bench;
+}
+
+Benchmark* VisualBenchmarkStream::innerNext() {
     while (fBenches) {
         Benchmark* bench = fBenches->factory()(NULL);
         fBenches = fBenches->next();