Title: [253825] trunk/LayoutTests
Revision
253825
Author
you...@apple.com
Date
2019-12-20 09:37:34 -0800 (Fri, 20 Dec 2019)

Log Message

webrtc/video-autoplay.html is flaky on iOS simulator
https://bugs.webkit.org/show_bug.cgi?id=205495

Reviewed by Eric Carlson.

* webrtc/video-autoplay-expected.txt:
* webrtc/video-autoplay.html:
Do test one video at a time so that one video does not interrupt the other.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (253824 => 253825)


--- trunk/LayoutTests/ChangeLog	2019-12-20 17:34:10 UTC (rev 253824)
+++ trunk/LayoutTests/ChangeLog	2019-12-20 17:37:34 UTC (rev 253825)
@@ -1,3 +1,14 @@
+2019-12-20  youenn fablet  <you...@apple.com>
+
+        webrtc/video-autoplay.html is flaky on iOS simulator
+        https://bugs.webkit.org/show_bug.cgi?id=205495
+
+        Reviewed by Eric Carlson.
+
+        * webrtc/video-autoplay-expected.txt:
+        * webrtc/video-autoplay.html:
+        Do test one video at a time so that one video does not interrupt the other.
+
 2019-12-20  Chris Dumez  <cdu...@apple.com>
 
         Unreviewed, mark imported/w3c/web-platform-tests/2dcontext/wide-gamut-canvas as flaky.

Modified: trunk/LayoutTests/webrtc/video-autoplay-expected.txt (253824 => 253825)


--- trunk/LayoutTests/webrtc/video-autoplay-expected.txt	2019-12-20 17:34:10 UTC (rev 253824)
+++ trunk/LayoutTests/webrtc/video-autoplay-expected.txt	2019-12-20 17:37:34 UTC (rev 253825)
@@ -1,10 +1,12 @@
  
+  
 
-
 PASS Ensuring autoplay does not work on regular video elements 
 PASS Ensuring autoplay does not work on regular video elements when getUserMedia is on 
 PASS Local media stream autoplay 
 PASS Remote media stream autoplay 
-PASS Granting getUserMedia should not start paused media 
+PASS Granting getUserMedia should not start paused media 1 
+PASS Granting getUserMedia should not start paused media 2 
+PASS Granting getUserMedia should not start paused media 3 
 PASS Granting getUserMedia should not start paused media or media that cannot autoplay due to autoplay restrictions 
 

Modified: trunk/LayoutTests/webrtc/video-autoplay.html (253824 => 253825)


--- trunk/LayoutTests/webrtc/video-autoplay.html	2019-12-20 17:34:10 UTC (rev 253824)
+++ trunk/LayoutTests/webrtc/video-autoplay.html	2019-12-20 17:37:34 UTC (rev 253825)
@@ -74,22 +74,31 @@
 }, "Remote media stream autoplay");
 
 promise_test(async (test) => {
-    if (window.internals) {
+    if (window.internals)
         internals.setMediaElementRestrictions(video1, "");
+
+    video1.src = "" "../media/content/audio-tracks");
+    await video1.play();
+
+    await video1.pause();
+
+    video4.srcObject = await navigator.mediaDevices.getUserMedia({ video : true });
+
+    while (video4.paused)
+        await new Promise(resolve => setTimeout(resolve, 50));
+
+    assert_true(video1.paused, "paused video");
+
+    video1.remove();
+}, "Granting getUserMedia should not start paused media 1");
+
+promise_test(async (test) => {
+    if (window.internals)
         internals.setMediaElementRestrictions(video2, "");
-        internals.setMediaElementRestrictions(video3, "");
-        internals.setMediaElementRestrictions(video4, "");
-    }
 
-    video1.src = "" "../media/content/audio-tracks");
     video2.src = "" "../media/content/audio-tracks");
-    video3.src = "" "../media/content/audio-tracks");
-
-    await video1.play();
     await video2.play();
-    await video3.play();
 
-    await video1.pause();
     let removedVideo2 = video2;
     removedVideo2.remove();
 
@@ -98,14 +107,25 @@
     while (video4.paused)
         await new Promise(resolve => setTimeout(resolve, 50));
 
-    assert_true(video1.paused, "paused video");
     assert_true(removedVideo2.paused, "out of DOM video");
+}, "Granting getUserMedia should not start paused media 2");
+
+promise_test(async (test) => {
+    if (window.internals)
+        internals.setMediaElementRestrictions(video3, "");
+
+    video3.src = "" "../media/content/audio-tracks");
+    await video3.play();
+
+    video4.srcObject = await navigator.mediaDevices.getUserMedia({ video : true });
+
+    while (video4.paused)
+        await new Promise(resolve => setTimeout(resolve, 50));
+
     assert_false(video3.paused, "still playing video");
 
-    video1.remove();
     video3.remove();
-    video4.remove();
-}, "Granting getUserMedia should not start paused media");
+}, "Granting getUserMedia should not start paused media 3");
 
 promise_test(async (test) => {
     if (window.internals) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to