Bug 1404187 - group timeout messages for MediaTestManager. draft
authorJW Wang <jwwang@mozilla.com>
Fri, 29 Sep 2017 11:25:31 +0800
changeset 672447 82dd307098918f727d3bc2ecbf28d34839280a7d
parent 672446 e39b2fbc3eeea67a5692793ebab389ee6bcb4d0f
child 733796 ed2416d60ca60b8b0581024e507771415dd111f7
push id82240
push userjwwang@mozilla.com
push dateFri, 29 Sep 2017 05:07:36 +0000
bugs1404187
milestone58.0a1
Bug 1404187 - group timeout messages for MediaTestManager. So different test timeouts from the same test case will fall under the same bug. MozReview-Commit-ID: LDstAhOpkYK
dom/media/test/manifest.js
--- a/dom/media/test/manifest.js
+++ b/dom/media/test/manifest.js
@@ -1685,17 +1685,18 @@ function MediaTestManager() {
   // Don't call more than once per token.
   this.started = function(token, handler) {
     this.tokens.push(token);
     this.numTestsRunning++;
     this.handlers[token] = handler;
 
     var onTimeout = () => {
       this.hasTimeout = true;
-      ok(false, `${token} timed out!`);
+      ok(false, "Test timed out!");
+      info(`${token} timed out!`);
       this.finished(token);
     };
     // Default timeout to 180s for each test.
     // Call SimpleTest._originalSetTimeout() to bypass the flaky timeout checker.
     this.timers[token] = SimpleTest._originalSetTimeout.call(window, onTimeout, 180000);
 
     is(this.numTestsRunning, this.tokens.length,
        "[started " + token + " t=" + elapsedTime(this.startTime) + "] Length of array should match number of running tests");