Bug 1293576: [MSE] P2. Fix mochitest. r?gerald draft
authorJean-Yves Avenard <jyavenard@mozilla.com>
Wed, 10 Aug 2016 16:21:36 +1000
changeset 398980 e43df1921b2dec90e0387843311add1aa228969e
parent 398979 604247b6c4a49122c64a2684c0964524843be953
child 398981 68ea8e5784b44beab867f9feba11125b386f89b1
push id25694
push userbmo:jyavenard@mozilla.com
push dateWed, 10 Aug 2016 06:24:12 +0000
reviewersgerald
bugs1293576
milestone51.0a1
Bug 1293576: [MSE] P2. Fix mochitest. r?gerald The mochitest relied that the video track was processed first. Additionally, change for the file with only a single video track as the previous video didn't have aligned segments, making the use of sequence mode useless. We swap the segment around, which allow to more easily visually inspect the result (counter goes forward and then back) MozReview-Commit-ID: 33PsrmRF1GL
dom/media/mediasource/test/test_Sequence_mp4.html
--- a/dom/media/mediasource/test/test_Sequence_mp4.html
+++ b/dom/media/mediasource/test/test_Sequence_mp4.html
@@ -17,23 +17,23 @@ runWithMSE(function(ms, v) {
     ok(true, "Receive a sourceopen event");
     ok(!receivedSourceOpen, "Should only receive one sourceopen for this test");
     receivedSourceOpen = true;
     var sb = ms.addSourceBuffer("video/mp4");
     ok(sb, "Create a SourceBuffer");
     sb.addEventListener('error', (e) => { ok(false, "Got Error: " + e); SimpleTest.finish(); });
     sb.mode = 'sequence';
 
-    fetchAndLoad(sb, 'bipbop/bipbop', ['init'], '.mp4')
-    .then(fetchAndLoad.bind(null, sb, 'bipbop/bipbop', ['2'], '.m4s'))
-    .then(fetchAndLoad.bind(null, sb, 'bipbop/bipbop', ['5'], '.m4s'))
+    fetchAndLoad(sb, 'bipbop/bipbop_video', ['init'], '.mp4')
+    .then(fetchAndLoad.bind(null, sb, 'bipbop/bipbop_video', ['5'], '.m4s'))
+    .then(fetchAndLoad.bind(null, sb, 'bipbop/bipbop_video', ['2'], '.m4s'))
     .then(function() {
         is(v.buffered.length, 1, "Continuous buffered range");
         is(v.buffered.start(0), 0, "Buffered range starts at 0");
-        ok(sb.timestampOffset < 0, "SourceBuffer.timestampOffset set to allow continuous range");
+        ok(sb.timestampOffset > 0, "SourceBuffer.timestampOffset set to allow continuous range");
         SimpleTest.finish();
       });
   });
 });
 </script>
 </pre>
 </body>
 </html>