commit d0530a6e035d6bbba42313b9e075672c08a89c32
author Austin Eng <enga@chromium.org> Tue Jan 18 19:40:35 2022 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Jan 18 19:40:35 2022 +0000
tree e668ebf6f275997ae9cc65aedb9758041c3dd495
parent 394c17a1bbbb59a43f60e23eff95266f0766f882

Use ErrorLog instead of InfoLog in InstanceBase::ConsumedError

Printing to stdout actually impacts Web Tests results which
makes managing expectations difficult, especially if logs
change.

Change InfoLog to ErrorLog:
 1) because this is ConsumedError, after all
 2) until we figure out if we can make Web Tests ignore stdout

Bug: chromium:1266550
Change-Id: I05f47f8b6d7a8e11568c5ee87eabab0cfd7c02d6
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/76680
Reviewed-by: Loko Kung <lokokung@google.com>
Commit-Queue: Austin Eng <enga@chromium.org>

src/dawn_native/Instance.cpp

diff --git a/src/dawn_native/Instance.cpp b/src/dawn_native/Instance.cpp
index 201487e..7b49351 100644
--- a/src/dawn_native/Instance.cpp
+++ b/src/dawn_native/Instance.cpp
@@ -345,7 +345,7 @@
             std::unique_ptr<ErrorData> error = maybeError.AcquireError();
 
             ASSERT(error != nullptr);
-            dawn::InfoLog() << error->GetFormattedMessage();
+            dawn::ErrorLog() << error->GetFormattedMessage();
             return true;
         }
         return false;