Bug 1294605 - Enable test code waiting for this bug. r=jib draft
authorAndreas Pehrson <pehrsons@gmail.com>
Mon, 22 Aug 2016 18:27:38 +0200
changeset 428039 53a16d3fac963070f943935437d062ca7b245ea9
parent 428038 9bf16d60ef7175e67eefa067ef57dba0d6c9d350
child 428040 5c3498938bf7f62e61f811b6fc8581c8e0ed3f42
push id33215
push userbmo:pehrson@telenordigital.com
push dateFri, 21 Oct 2016 13:47:09 +0000
reviewersjib
bugs1294605
milestone52.0a1
Bug 1294605 - Enable test code waiting for this bug. r=jib MozReview-Commit-ID: 3TV58pT1SF2
dom/media/tests/mochitest/test_getUserMedia_addtrack_removetrack_events.html
dom/media/tests/mochitest/test_getUserMedia_mediaStreamTrackClone.html
--- a/dom/media/tests/mochitest/test_getUserMedia_addtrack_removetrack_events.html
+++ b/dom/media/tests/mochitest/test_getUserMedia_addtrack_removetrack_events.html
@@ -45,19 +45,18 @@ runTest(() => getUserMedia({audio: true,
     stream.addEventListener("removetrack", function onRemovetrack(event) {
       ok(false, "removetrack fired unexpectedly for track " + event.track.id);
     });
 
     return getUserMedia({audio: true, video: true});
   })
   .then(s => {
     newStream = s;
-// TODO: Uncomment once bug 1294605 is fixed
-//    info("Stopping an original track");
-//    stopTrack(stream.getTracks()[0]);
+    info("Stopping an original track");
+    stopTrack(stream.getTracks()[0]);
 
     return spinEventLoop();
   })
   .then(() => {
     info("Removing original tracks");
     stream.getTracks().forEach(t => (stream.removeTrack(t), tracks.push(t)));
 
     return spinEventLoop();
--- a/dom/media/tests/mochitest/test_getUserMedia_mediaStreamTrackClone.html
+++ b/dom/media/tests/mochitest/test_getUserMedia_mediaStreamTrackClone.html
@@ -49,20 +49,19 @@
       cloneStream.addTrack(inceptionClone);
 
       // cloneStream is now essentially the same as stream.clone();
       checkMediaStreamCloneAgainstOriginal(cloneStream, stream);
 
       var test = createMediaElement('video', 'testClonePlayback');
       var playback = new MediaStreamPlayback(test, cloneStream);
       return playback.playMedia(false).then(() => {
-// TODO: Uncomment once bug 1294605 is fixed
-//          info("Testing that clones of ended tracks are ended");
-//          cloneStream.clone().getTracks().forEach(t =>
-//            is(t.readyState, "ended", "Track " + t.id + " should be ended"));
+          info("Testing that clones of ended tracks are ended");
+          cloneStream.clone().getTracks().forEach(t =>
+            is(t.readyState, "ended", "Track " + t.id + " should be ended"));
         })
         .then(() => {
           clones.forEach(t => t.stop());
           track.stop();
         });
     })
     .then(() => getUserMedia({audio: true, video: true})).then(stream => {
       info("Test adding many track clones to the original stream");