Title: [286194] trunk/LayoutTests
Revision
286194
Author
commit-qu...@webkit.org
Date
2021-11-28 10:19:19 -0800 (Sun, 28 Nov 2021)

Log Message

Weird JS usage in webrtc/multi-video.html
https://bugs.webkit.org/show_bug.cgi?id=233540

Patch by Philippe Normand <pnorm...@igalia.com> on 2021-11-28
Reviewed by Youenn Fablet.

* webrtc/multi-video.html: MediaStream.getVideoTracks() accepts no argument and returns a
sequence. The MediaStream constructor needs a sequence of tracks.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (286193 => 286194)


--- trunk/LayoutTests/ChangeLog	2021-11-28 04:28:11 UTC (rev 286193)
+++ trunk/LayoutTests/ChangeLog	2021-11-28 18:19:19 UTC (rev 286194)
@@ -1,3 +1,13 @@
+2021-11-28  Philippe Normand  <pnorm...@igalia.com>
+
+        Weird JS usage in webrtc/multi-video.html
+        https://bugs.webkit.org/show_bug.cgi?id=233540
+
+        Reviewed by Youenn Fablet.
+
+        * webrtc/multi-video.html: MediaStream.getVideoTracks() accepts no argument and returns a
+        sequence. The MediaStream constructor needs a sequence of tracks.
+
 2021-11-27  Sam Weinig  <wei...@apple.com>
 
         [CSS Color 4] Add support for oklab() and oklch() colors

Modified: trunk/LayoutTests/webrtc/multi-video.html (286193 => 286194)


--- trunk/LayoutTests/webrtc/multi-video.html	2021-11-28 04:28:11 UTC (rev 286193)
+++ trunk/LayoutTests/webrtc/multi-video.html	2021-11-28 18:19:19 UTC (rev 286194)
@@ -71,12 +71,13 @@
             setTimeout(() => reject("Test timed out"), 5000);
         });
     }).then((stream) => {
-        video1.srcObject = new MediaStream(stream.getVideoTracks([0]));
-        video2.srcObject = new MediaStream(stream.getVideoTracks([1]));
-        video3.srcObject = new MediaStream(stream.getVideoTracks([2]));
-        video4.srcObject = new MediaStream(stream.getVideoTracks([3]));
-        video5.srcObject = new MediaStream(stream.getVideoTracks([4]));
-        video6.srcObject = new MediaStream(stream.getVideoTracks([5]));
+        var tracks = stream.getVideoTracks();
+        video1.srcObject = new MediaStream([tracks[0]]);
+        video2.srcObject = new MediaStream([tracks[1]]);
+        video3.srcObject = new MediaStream([tracks[2]]);
+        video4.srcObject = new MediaStream([tracks[3]]);
+        video5.srcObject = new MediaStream([tracks[4]]);
+        video6.srcObject = new MediaStream([tracks[5]]);
         return Promise.all([video1.play(), video2.play(), video3.play(), video4.play(), video5.play(), video6.play()]);
     }).then(() => {
         return waitFor(100);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to