ok, do not print time or memory use for skipped tests

Change-Id: I6feb4924ab67e1da08f923315ef6d3a580881b23
Reviewed-on: https://skia-review.googlesource.com/26340
Reviewed-by: Mike Klein <mtklein@chromium.org>
Commit-Queue: Mike Klein <mtklein@chromium.org>

1 files changed, 8 insertions, 4 deletions

diff --git a/tools/ok_vias.cpp b/tools/ok_vias.cpp
index 37d371f797..b7fd2fef13 100644
--- a/tools/ok_vias.cpp
+++ b/tools/ok_vias.cpp
@@ -148,8 +148,10 @@ struct Time : Dst {
         std::chrono::duration<double, std::milli> elapsed = std::chrono::steady_clock::now()
                                                           - start;
 
-        auto msg = HumanizeMs(elapsed.count());
-        ok_log(msg.c_str());
+        if (status != Status::Skipped) {
+            auto msg = HumanizeMs(elapsed.count());
+            ok_log(msg.c_str());
+        }
         return status;
     }
 
@@ -171,8 +173,10 @@ struct Memory : Dst {
     Status draw(Src* src) override {
         Status status = target->draw(src);
 
-        auto msg = SkStringPrintf("%dMB", sk_tools::getMaxResidentSetSizeMB());
-        ok_log(msg.c_str());
+        if (status != Status::Skipped) {
+            auto msg = SkStringPrintf("%dMB", sk_tools::getMaxResidentSetSizeMB());
+            ok_log(msg.c_str());
+        }
 
         return status;
     }