Bug 1275461: added missing test failure in case of timeout draft
authorNils Ohlmeier [:drno] <drno@ohlmeier.org>
Mon, 22 Aug 2016 16:01:01 -0700
changeset 404127 5d53df55b13d778de9167389c69346396257270f
parent 404063 3345f0e8ed3bd33d8af6e2b9bf1fd8bf397a45bc
child 529102 720093202d006f5dc15bc6f34f8303a5fffa692a
push id27120
push userdrno@ohlmeier.org
push dateMon, 22 Aug 2016 23:01:23 +0000
bugs1275461
milestone51.0a1
Bug 1275461: added missing test failure in case of timeout MozReview-Commit-ID: F0cqAm1x5em
dom/media/tests/mochitest/head.js
--- a/dom/media/tests/mochitest/head.js
+++ b/dom/media/tests/mochitest/head.js
@@ -843,17 +843,20 @@ VideoStreamHelper.prototype = {
     return addFinallyToPromise(timeout(
       Promise.all([
         this._helper.waitForPixelColor(canvas, this._helper.green, 128,
                                        canvas.id + " should become green"),
         this._helper.waitForPixelColor(canvas, this._helper.red, 128,
                                        canvas.id + " should become red")
       ]),
       timeout_value,
-      "Timed out waiting for frames")).finally(() => clearInterval(intervalId));
+      "Timed out waiting for frames")).finally(() => {
+        ok(false, "Waiting for red and green timed out");
+        clearInterval(intervalId);
+        });
   },
 
   verifyNoFrames: function(canvas) {
     return this.waitForFrames(canvas).then(
       () => ok(false, "Color should not change"),
       () => ok(true, "Color should not change")
     );
   }