[webkit-changes] [287078] trunk/LayoutTests

Wed, 15 Dec 2021 09:27:04 -0800

Title: [287078] trunk/LayoutTests
Revision
287078
Author
you...@apple.com
Date
2021-12-15 09:26:13 -0800 (Wed, 15 Dec 2021)

Log Message

[ Mac wk2 and iOS wk2 ] webrtc/libwebrtc/descriptionGetters.html is flaky failing.
https://bugs.webkit.org/show_bug.cgi?id=214084
<rdar://problem/65225336>

Reviewed by Eric Carlson.

* platform/ios-wk2/TestExpectations:
* platform/mac-wk2/TestExpectations:
* webrtc/libwebrtc/descriptionGetters.html:
We are no longer getting the descriptions straight from the backend.
Instead, we are storing them at end of createOffer or answer/set descriptions time.
Update test accordingly.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (287077 => 287078)


--- trunk/LayoutTests/ChangeLog	2021-12-15 17:10:36 UTC (rev 287077)
+++ trunk/LayoutTests/ChangeLog	2021-12-15 17:26:13 UTC (rev 287078)
@@ -1,3 +1,18 @@
+2021-12-15  Youenn Fablet  <you...@apple.com>
+
+        [ Mac wk2 and iOS wk2 ] webrtc/libwebrtc/descriptionGetters.html is flaky failing.
+        https://bugs.webkit.org/show_bug.cgi?id=214084
+        <rdar://problem/65225336>
+
+        Reviewed by Eric Carlson.
+
+        * platform/ios-wk2/TestExpectations:
+        * platform/mac-wk2/TestExpectations:
+        * webrtc/libwebrtc/descriptionGetters.html:
+        We are no longer getting the descriptions straight from the backend.
+        Instead, we are storing them at end of createOffer or answer/set descriptions time.
+        Update test accordingly.
+
 2021-12-15  Jer Noble  <jer.no...@apple.com>
 
         [ iOS EWS ]  2X media/track (layout-tests) are a constant image failure

Modified: trunk/LayoutTests/platform/ios-wk2/TestExpectations (287077 => 287078)


--- trunk/LayoutTests/platform/ios-wk2/TestExpectations	2021-12-15 17:10:36 UTC (rev 287077)
+++ trunk/LayoutTests/platform/ios-wk2/TestExpectations	2021-12-15 17:26:13 UTC (rev 287078)
@@ -1895,8 +1895,6 @@
 
 webkit.org/b/21365 imported/w3c/web-platform-tests/html/semantics/embedded-content/the-video-element/intrinsic_sizes.htm [ Pass Failure ]
 
-webkit.org/b/214084 webrtc/libwebrtc/descriptionGetters.html [ Pass Failure ]
-
 webkit.org/b/214322 [ Debug ] http/tests/workers/worker-importScripts-banned-mimetype.html [ Pass Crash ]
 
 webkit.org/b/214382 [ Release ] imported/w3c/web-platform-tests/websockets/remove-own-iframe-during-onerror.window.html [ Pass Failure ]

Modified: trunk/LayoutTests/platform/mac-wk2/TestExpectations (287077 => 287078)


--- trunk/LayoutTests/platform/mac-wk2/TestExpectations	2021-12-15 17:10:36 UTC (rev 287077)
+++ trunk/LayoutTests/platform/mac-wk2/TestExpectations	2021-12-15 17:26:13 UTC (rev 287078)
@@ -1109,8 +1109,6 @@
 
 webkit.org/b/229076 [ BigSur Debug ] webrtc/video-interruption.html [ Pass Crash ]
 
-webkit.org/b/214084 webrtc/libwebrtc/descriptionGetters.html [ Pass Failure ]
-
 webkit.org/b/213804 fast/scrolling/mac/scroll-snapping-in-progress.html [ Pass Failure ]
 
 webkit.org/b/229206 [ Debug ] http/wpt/webrtc/sframe-transform-error.html [ Pass Failure ]

Modified: trunk/LayoutTests/webrtc/libwebrtc/descriptionGetters.html (287077 => 287078)


--- trunk/LayoutTests/webrtc/libwebrtc/descriptionGetters.html	2021-12-15 17:10:36 UTC (rev 287077)
+++ trunk/LayoutTests/webrtc/libwebrtc/descriptionGetters.html	2021-12-15 17:26:13 UTC (rev 287078)
@@ -28,57 +28,61 @@
 
         localConnection.addTrack(stream.getVideoTracks()[0], stream);
         return localConnection.createOffer();
-    }).then((desc) => {
+    }).then(async (desc) => {
         currentSDP = desc.sdp;
 
-        assert_equals(localConnection.currentLocalDescription, null);
-        assert_equals(localConnection.pendingLocalDescription, null);
-        assert_equals(localConnection.localDescription, null);
+        assert_equals(localConnection.currentLocalDescription, null, "test 1");
+        assert_equals(localConnection.pendingLocalDescription, null, "test 2");
+        assert_equals(localConnection.localDescription, null, "test 3");
 
-        assert_equals(remoteConnection.currentRemoteDescription, null);
-        assert_equals(remoteConnection.pendingRemoteDescription, null);
-        assert_equals(remoteConnection.remoteDescription, null);
+        assert_equals(remoteConnection.currentRemoteDescription, null, "test 4");
+        assert_equals(remoteConnection.pendingRemoteDescription, null, "test 5");
+        assert_equals(remoteConnection.remoteDescription, null, "test 6");
 
         localDescriptionPromise = localConnection.setLocalDescription(desc);
-        remoteDescriptionPromise = remoteConnection.setRemoteDescription(desc);
+        assert_equals(localConnection.currentLocalDescription, null, "test 7");
+        assert_equals(localConnection.pendingLocalDescription, null, "test 8");
 
-        assert_equals(localConnection.currentLocalDescription, null, "test 1");
-        assert_equals(sdpWithoutCandidate(localConnection.pendingLocalDescription.sdp), sdpWithoutCandidate(currentSDP), "test 2");
-        assert_equals(sdpWithoutCandidate(localConnection.localDescription.sdp), sdpWithoutCandidate(currentSDP), "test 3");
+        await localDescriptionPromise;
+        assert_equals(sdpWithoutCandidate(localConnection.pendingLocalDescription.sdp), sdpWithoutCandidate(currentSDP), "test 9");
+        assert_equals(sdpWithoutCandidate(localConnection.localDescription.sdp), sdpWithoutCandidate(currentSDP), "test 10");
 
-        assert_equals(remoteConnection.currentRemoteDescription, null);
-        assert_equals(remoteConnection.pendingRemoteDescription.sdp, currentSDP, "test 4");
-        assert_equals(remoteConnection.remoteDescription.sdp, currentSDP, "test 5");
+        remoteDescriptionPromise = remoteConnection.setRemoteDescription(desc);
+        assert_equals(remoteConnection.currentLocalDescription, null, "test 11");
+        assert_equals(remoteConnection.pendingLocalDescription, null, "test 12");
 
-        return Promise.all([localDescriptionPromise, remoteDescriptionPromise]);
+        await remoteDescriptionPromise;
+        assert_equals(remoteConnection.currentRemoteDescription, null, "test 13");
+        assert_equals(remoteConnection.pendingRemoteDescription.sdp, currentSDP, "test 14");
+        assert_equals(remoteConnection.remoteDescription.sdp, currentSDP, "test 15");
     }).then(() => {
         return remoteConnection.createAnswer();
-    }).then((desc) => {
-        assert_equals(localConnection.currentLocalDescription, null);
-        assert_equals(localConnection.localDescription.sdp, localConnection.pendingLocalDescription.sdp, "test 7");
+    }).then(async (desc) => {
+        const remoteDescriptionPromise = localConnection.setRemoteDescription(desc);
+        assert_equals(localConnection.currentRemoteDescription, null, "test 16");
+        assert_equals(localConnection.pendingRemoteDescription, null, "test 17");
 
-        localConnection.setRemoteDescription(desc);
+        await remoteDescriptionPromise;
+        assert_equals(localConnection.pendingRemoteDescription, null, "test 18");
+        assert_not_equals(localConnection.currentRemoteDescription, null, "test 19");
 
-        assert_equals(localConnection.currentLocalDescription.sdp, localConnection.localDescription.sdp, "test 8");
-        assert_equals(localConnection.pendingLocalDescription, null);
+        const localDescriptionPromise = remoteConnection.setLocalDescription(desc);
+        assert_equals(remoteConnection.currentLocalDescription, null, "test 20");
+        assert_equals(remoteConnection.pendingLocalDescription, null, "test 21");
 
-        assert_equals(remoteConnection.currentRemoteDescription, null);
-        assert_equals(remoteConnection.pendingRemoteDescription.sdp, remoteConnection.remoteDescription.sdp, "test 10");
-
-        promise = remoteConnection.setLocalDescription(desc);
-
-        assert_equals(remoteConnection.pendingRemoteDescription, null);
-        return promise;
+        await localDescriptionPromise;
+        assert_not_equals(remoteConnection.currentLocalDescription, null, "test 22");
+        assert_equals(remoteConnection.pendingLocalDescription, null, "test 23");
     }).then(() => {
         desc = remoteConnection.currentRemoteDescription;
 
         localConnection.createDataChannel("test");
 
-        assert_equals(localConnection.currentLocalDescription.sdp, localConnection.localDescription.sdp, "test 14");
-        assert_equals(localConnection.pendingLocalDescription, null);
+        assert_equals(localConnection.currentLocalDescription.sdp, localConnection.localDescription.sdp, "test 24");
+        assert_equals(localConnection.pendingLocalDescription, null, "test 25");
 
-        assert_equals(remoteConnection.currentRemoteDescription.sdp, remoteConnection.remoteDescription.sdp, "test 16");
-        assert_equals(remoteConnection.pendingRemoteDescription, null);
+        assert_equals(remoteConnection.currentRemoteDescription.sdp, remoteConnection.remoteDescription.sdp, "test 26");
+        assert_equals(remoteConnection.pendingRemoteDescription, null, "test 27");
     });
 }, "description getters when changing description from video to video & data channel");
         </script>

_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to