[webkit-changes] [WebKit/WebKit] 523cb7: REGRESSION(263499 at main): [GStreamer] Several webrt...
Philippe Normand
noreply at github.com
Tue May 2 04:02:15 PDT 2023
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 523cb7fde9c6422fb077a53a9b58ea07e9a18a1e
https://github.com/WebKit/WebKit/commit/523cb7fde9c6422fb077a53a9b58ea07e9a18a1e
Author: Philippe Normand <philn at igalia.com>
Date: 2023-05-02 (Tue, 02 May 2023)
Changed paths:
M LayoutTests/platform/glib/TestExpectations
M Source/WebCore/platform/mediastream/RealtimeMediaSource.h
M Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp
M Source/WebCore/platform/mediastream/gstreamer/RealtimeIncomingSourceGStreamer.cpp
M Source/WebCore/platform/mediastream/gstreamer/RealtimeIncomingSourceGStreamer.h
Log Message:
-----------
REGRESSION(263499 at main): [GStreamer] Several webrtc tests timing out
https://bugs.webkit.org/show_bug.cgi?id=256127
Reviewed by Xabier Rodriguez-Calvar.
The timeouts were caused by the playback appsrc forwarding caps queries to the incoming source. Also
the incoming source shouldn't forward stream collection events to the playback appsrc because it is
already handling stream collections itself.
Driving by, the handleDownstreamEvent plumbing is no longer needed now that we know which appsrc is
related to the incoming source appsinks.
* LayoutTests/platform/glib/TestExpectations:
* Source/WebCore/platform/mediastream/RealtimeMediaSource.h:
* Source/WebCore/platform/mediastream/gstreamer/GStreamerMediaStreamSource.cpp:
* Source/WebCore/platform/mediastream/gstreamer/RealtimeIncomingSourceGStreamer.cpp:
(WebCore::RealtimeIncomingSourceGStreamer::RealtimeIncomingSourceGStreamer):
(WebCore::RealtimeIncomingSourceGStreamer::registerClient):
(WebCore::RealtimeIncomingSourceGStreamer::handleDownstreamEvent): Deleted.
* Source/WebCore/platform/mediastream/gstreamer/RealtimeIncomingSourceGStreamer.h:
(WebCore::RealtimeIncomingSourceGStreamer::dispatchSample):
Canonical link: https://commits.webkit.org/263581@main
More information about the webkit-changes
mailing list