commit e8e9d5daf59fcf19f1a2e1b36e8da7437e0264f5
author Corentin Wallez <cwallez@chromium.org> Wed Dec 11 11:10:01 2019 +0000
committer Commit Bot service account <commit-bot@chromium.org> Wed Dec 11 11:10:01 2019 +0000
tree a3a1577373d1d4dbedbf505c95e06a1f9b1be79d
parent ea89189d336d691cd5661f81e1a35357fd346a5e

Use InfoLog to report perftest results.

On Android printing to stdout doesn't show in logcat. Use InfoLog
instead of fprintf(stdout) so that the message get routed to
android_log.

BUG=dawn:286

Change-Id: I067a12d8a790e42d75617c760a2693a3d94ebda6
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/14481
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Kai Ninomiya <kainino@chromium.org>

src/tests/perf_tests/DawnPerfTest.cpp

diff --git a/src/tests/perf_tests/DawnPerfTest.cpp b/src/tests/perf_tests/DawnPerfTest.cpp
index fd4db8a..2971a4d 100644
--- a/src/tests/perf_tests/DawnPerfTest.cpp
+++ b/src/tests/perf_tests/DawnPerfTest.cpp
@@ -385,10 +385,8 @@
 
     // The results are printed according to the format specified at
     // [chromium]//build/scripts/slave/performance_log_processor.py
-    fflush(stdout);
-    printf("%sRESULT %s%s: %s= %s%f%s %s\n", important ? "*" : "", testSuite, testName,
-           trace.c_str(), "", value, "", units.c_str());
-    fflush(stdout);
+    dawn::InfoLog() << (important ? "*" : "") << "RESULT " << testSuite << testName << ": " << trace
+                    << "= " << value << " " << units;
 }
 
 void DawnPerfTestBase::PrintResult(const std::string& trace,
@@ -403,8 +401,6 @@
 
     // The results are printed according to the format specified at
     // [chromium]//build/scripts/slave/performance_log_processor.py
-    fflush(stdout);
-    printf("%sRESULT %s%s: %s= %s%u%s %s\n", important ? "*" : "", testName, testSuite,
-           trace.c_str(), "", value, "", units.c_str());
-    fflush(stdout);
+    dawn::InfoLog() << (important ? "*" : "") << "RESULT " << testSuite << testName << ": " << trace
+                    << "= " << value << " " << units;
 }