Issue 6822 - Report browser test errors

test/runners/chromium_remote_process.js

Index: test/runners/chromium_remote_process.js
===================================================================
--- a/test/runners/chromium_remote_process.js
+++ b/test/runners/chromium_remote_process.js
@@ -174,16 +174,25 @@
       if (callResult.exceptionDetails)
         throwException(callResult.exceptionDetails, scriptName);
 
       let promiseResult = await Runtime.awaitPromise({
         promiseObjectId: callResult.result.objectId
       });
       if (promiseResult.exceptionDetails)
         throwException(promiseResult.exceptionDetails, scriptName);
+
+      return Runtime.evaluate({
+        expression: "window.consoleLogs",
+        returnByValue: true
+      }).then(result =>
+      {
+        if (result.result.value.failures != 0)

[Sebastian Noack, 2018/09/17 16:18:06]

I'm not sure if I understand correctly, but wouldn't we want to log the failure
messages themselves rather than just seeing "Test failure" if any browser error
occurred?

[hub, 2018/09/18 22:23:36]

the messages ARE displayed aleready. Have always been. This is just for
propagating the error back up.

+          return Promise.reject("Test failure");
+      });
     }
     finally
     {
       client.close();
     }
   });
 }