Skip to content

REGRESSION(309987@main): [GStreamer] ASSERT when running imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-relay-canvas.https.html#62199

Merged
webkit-commit-queue merged 1 commit into
WebKit:mainfrom
philn:eng/311651
Apr 8, 2026

Conversation

@philn
Copy link
Copy Markdown
Member

@philn philn commented Apr 7, 2026

49a1bdb

REGRESSION(309987@main): [GStreamer] ASSERT when running imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-relay-canvas.https.html
https://bugs.webkit.org/show_bug.cgi?id=311651

Reviewed by Xabier Rodriguez-Calvar.

Restore the PeerConnectionBackend code removed in 309987@main for GStreamer ports, until bug 311652
is fixed.

* Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp:
(WebCore::PeerConnectionBackend::setRemoteDescriptionSucceeded):

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

c5ef28c

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 win
✅ 🧪 bindings ✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 ✅ 🧪 win-tests
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt ✅ 🧪 api-mac-debug ✅ 🛠 gtk3-libwebrtc
✅ 🧪 api-ios ✅ 🧪 mac-wk1 ✅ 🛠 gtk
✅ 🛠 ios-safer-cpp ✅ 🧪 mac-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 api-gtk
✅ 🛠 🧪 merge ✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🛠 playstation
✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2
✅ 🛠 tv ✅ 🛠 mac-safer-cpp
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@philn philn self-assigned this Apr 7, 2026
@philn philn added the Platform Portability improvements and other general platform improvements not driven directly by site bugs. label Apr 7, 2026
@philn philn requested a review from calvaris April 7, 2026 15:11
@philn
Copy link
Copy Markdown
Member Author

philn commented Apr 7, 2026

cc @csaavedra

@csaavedra
Copy link
Copy Markdown
Member

Should the expectation be updated/removed?

LayoutTests/platform/glib/TestExpectations:imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-relay-canvas.https.html [ Pass Failure ]

@csaavedra
Copy link
Copy Markdown
Member

Should the expectation be updated/removed?

LayoutTests/platform/glib/TestExpectations:imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-relay-canvas.https.html [ Pass Failure ]

Ah, this is about the assertion only, nevermind.

@philn philn added the merge-queue Applied to send a pull request to merge-queue label Apr 8, 2026
…/web-platform-tests/webrtc/RTCPeerConnection-relay-canvas.https.html

https://bugs.webkit.org/show_bug.cgi?id=311651

Reviewed by Xabier Rodriguez-Calvar.

Restore the PeerConnectionBackend code removed in 309987@main for GStreamer ports, until bug 311652
is fixed.

* Source/WebCore/Modules/mediastream/PeerConnectionBackend.cpp:
(WebCore::PeerConnectionBackend::setRemoteDescriptionSucceeded):

Canonical link: https://commits.webkit.org/310767@main
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 310767@main (49a1bdb): https://commits.webkit.org/310767@main

Reviewed commits have been landed. Closing PR #62199 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 49a1bdb into WebKit:main Apr 8, 2026
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Apr 8, 2026
@philn philn deleted the eng/311651 branch April 8, 2026 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Platform Portability improvements and other general platform improvements not driven directly by site bugs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants