Bug 1166832 - Remove usage of '!!' idiom. draft
authorMichael Froman <mfroman@mozilla.com>
Wed, 13 Jan 2016 18:07:17 -0600
changeset 321547 7175c404ba5ab08ffa23a272a3b7c0a8dbe1bcd9
parent 321546 d12fa9c3bb6acf28e4414a8640f0ab2a0bdf5240
child 321548 2e15b57fcc6593fcc5bf273320cdbfa6387c979e
push id9416
push usermfroman@nostrum.com
push dateThu, 14 Jan 2016 03:00:43 +0000
bugs1166832
milestone46.0a1
Bug 1166832 - Remove usage of '!!' idiom.
dom/media/tests/mochitest/test_peerConnection_multiple_captureStream_canvas_2d.html
--- a/dom/media/tests/mochitest/test_peerConnection_multiple_captureStream_canvas_2d.html
+++ b/dom/media/tests/mochitest/test_peerConnection_multiple_captureStream_canvas_2d.html
@@ -47,30 +47,30 @@ runNetworkTest(() => {
   test.chain.append([
     function FIND_REMOTE_VIDEO() {
       // we don't know which remote video is created first, so we don't know
       // which corresponds to the original local video (meaning we don't know
       // which color is expected in which remote
       var testremote = document.getElementById('pcRemote_remote3_video');
       ok(!testremote, "Should not have remote3 video element for pcRemote");
       testremote = document.getElementById('pcRemote_remote1_video');
-      ok(!!testremote, "Should have remote1 video element for pcRemote");
+      ok(testremote, "Should have remote1 video element for pcRemote");
 
       if (h.isPixel(h.getPixel(testremote), h.green, 128)) {
         info("testremote has green");
         vremote1 = testremote;
         vremote2 = document.getElementById('pcRemote_remote2_video');;
       } else if (h.isPixel(h.getPixel(testremote), h.blue, 128)) {
         info("testremote has blue");
         vremote2 = testremote;
         vremote1 = document.getElementById('pcRemote_remote2_video');
       }
 
-      ok(!!vremote1, "Should have remote1 video element for pcRemote");
-      ok(!!vremote2, "Should have remote2 video element for pcRemote");
+      ok(vremote1, "Should have remote1 video element for pcRemote");
+      ok(vremote2, "Should have remote2 video element for pcRemote");
     },
     function WAIT_FOR_REMOTE1_GREEN() {
       return h.waitForPixelColor(vremote1, h.green, 128,
                                  "pcRemote's remote1 should become green");
     },
     function WAIT_FOR_REMOTE2_BLUE() {
       return h.waitForPixelColor(vremote2, h.blue, 128,
                                  "pcRemote's remote2 should become green");