REGRESSION(272844@main): [ wk2 ] imported/w3c/web-platform-tests/webr…

…tc/RTCPeerConnection-GC.https.html is consistently failing.

https://bugs.webkit.org/show_bug.cgi?id=267355
rdar://120798707

Reviewed by Eric Carlson.

The previous try to fix the flakiness was not taking into account the case of resize event firing to handle the transition from (0,0) to (160,160).
If it fires while onVideoChange is running, onVideoChange would spin as resizeEvent promise would not be recomputed and stay resolved.

We simply remove the resize event check and only rely on the setTimeout check which has the same frequency as the video frame emition by the canvas.

* LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-GC.https.html:
* LayoutTests/platform/wk2/TestExpectations:

Canonical link: https://commits.webkit.org/272990@main

LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-GC.https.html

@@ -40,15 +40,11 @@
     const start = performance.now();
     const width = destVideo.videoWidth;
     const height = destVideo.videoHeight;
-    const resizeEvent = new Promise(r => destVideo.onresize = r);
     while (destVideo.videoWidth == width && destVideo.videoHeight == height) {
       if (performance.now() - start > 5000) {
         throw new Error("Timeout waiting for video size change");
       }
-      await Promise.race([
-        resizeEvent,
-        new Promise(r => setTimeout(r, 50)),
-      ]);
+      await new Promise(r => t.step_timeout(r, 50));
     }
   };
 

LayoutTests/platform/wk2/TestExpectations

@@ -884,5 +884,3 @@ webkit.org/b/262157 imported/w3c/web-platform-tests/css/css-contain/content-visi
 webkit.org/b/261314 [ Debug ] fast/mediastream/captureStream/canvas3d.html [ Pass Failure ]
 
 webkit.org/b/265700 webrtc/multi-video.html [ Pass Failure ]
-
-webkit.org/b/267355 imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-GC.https.html [ Pass Failure ]