Title: [289196] trunk/LayoutTests
Revision
289196
Author
[email protected]
Date
2022-02-06 20:18:01 -0800 (Sun, 06 Feb 2022)

Log Message

media/media-source/media-webm-vorbis-partial.html is failing
https://bugs.webkit.org/show_bug.cgi?id=236211
rdar://88298267

Reviewed by Darin Adler.

The manifest data was incorrect, the webm cluster size is 10004 bytes long.
Expand the test so it's consistent with the webm/opus one.

* media/media-source/content/test-vorbis-manifest.json:
* media/media-source/media-webm-vorbis-partial.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (289195 => 289196)


--- trunk/LayoutTests/ChangeLog	2022-02-07 03:58:03 UTC (rev 289195)
+++ trunk/LayoutTests/ChangeLog	2022-02-07 04:18:01 UTC (rev 289196)
@@ -1,3 +1,17 @@
+2022-02-06  Jean-Yves Avenard  <[email protected]>
+
+        media/media-source/media-webm-vorbis-partial.html is failing
+        https://bugs.webkit.org/show_bug.cgi?id=236211
+        rdar://88298267
+
+        Reviewed by Darin Adler.
+
+        The manifest data was incorrect, the webm cluster size is 10004 bytes long.
+        Expand the test so it's consistent with the webm/opus one.
+
+        * media/media-source/content/test-vorbis-manifest.json:
+        * media/media-source/media-webm-vorbis-partial.html:
+
 2022-02-06  Sam Weinig  <[email protected]>
 
         Remove duplicate tests that have been upstreamed to WPT

Modified: trunk/LayoutTests/media/media-source/content/test-vorbis-manifest.json (289195 => 289196)


--- trunk/LayoutTests/media/media-source/content/test-vorbis-manifest.json	2022-02-07 03:58:03 UTC (rev 289195)
+++ trunk/LayoutTests/media/media-source/content/test-vorbis-manifest.json	2022-02-07 04:18:01 UTC (rev 289196)
@@ -4,6 +4,6 @@
     "init": { "offset": 0, "size": 3744 },
     "duration": 1.33,
     "media": [
-        { "offset": 3744, "size": 9992, "timestamp": 0, "duration": 1.33 }
+        { "offset": 3744, "size": 10004, "timestamp": 0, "duration": 1.33 }
     ]
 }
\ No newline at end of file

Modified: trunk/LayoutTests/media/media-source/media-webm-vorbis-partial.html (289195 => 289196)


--- trunk/LayoutTests/media/media-source/media-webm-vorbis-partial.html	2022-02-07 03:58:03 UTC (rev 289195)
+++ trunk/LayoutTests/media/media-source/media-webm-vorbis-partial.html	2022-02-07 04:18:01 UTC (rev 289196)
@@ -16,8 +16,6 @@
         });
     }
 
-    // Append a vorbis webm with a single incomplete cluster. Ensure that content that is less
-    // than 2s in duration is properly demuxed.
     window.addEventListener('load', async event => {
         try {
             findMediaElement();
@@ -29,15 +27,26 @@
             await waitFor(source, 'sourceopen');
             waitFor(video, 'error').then(failTest);
 
-            run('source.duration = loader.duration()');
             run('sourceBuffer = source.addSourceBuffer(loader.type())');
             run('sourceBuffer.appendBuffer(loader.initSegment())');
             await waitFor(sourceBuffer, 'update');
 
-            consoleWrite('Append a media segment.')
-            run('sourceBuffer.appendBuffer(loader.mediaSegment(0))');
+            consoleWrite('Divide the first media segment in two.');
+            run('partial1 = loader.mediaSegment(0).slice(0, loader.mediaSegment(0).byteLength / 2)');
+            run('partial2 = loader.mediaSegment(0).slice(loader.mediaSegment(0).byteLength / 2)');
+
+            consoleWrite('Append a partial media segment.')
+            run('sourceBuffer.appendBuffer(partial1)');
             await waitFor(sourceBuffer, 'update');
+            testExpected('sourceBuffer.buffered.length', '1');
+            testExpected('sourceBuffer.buffered.end(0).toFixed(2)', '0.64', '==');
 
+            consoleWrite('Complete the partial media segment.')
+            run('sourceBuffer.appendBuffer(partial2)');
+            await waitFor(sourceBuffer, 'update');
+
+            testExpected('sourceBuffer.buffered.length', '1');
+            testExpected('sourceBuffer.buffered.end(0).toFixed(2)', '1.34', '==');
             testExpected('sourceBuffer.buffered.end(0) == source.duration', true);
 
             endTest();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to