webrtcbin: Remove queue after rtpfunnel
authorJan Schmidt <jan@centricular.com>
Sat, 19 Nov 2022 08:22:17 +0000 (19:22 +1100)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Sat, 19 Nov 2022 10:31:50 +0000 (10:31 +0000)
commitdfb5e3365ebc785abffe5cd9b416118c48252cb6
tree6fbe1f9aac19327186fe70529918f3d449bd4a0a
parent81775882509c6db43e29aa63fcd2ad86f877ff94
webrtcbin: Remove queue after rtpfunnel

The original BUNDLE support commit placed a queue after the
rtpfunnel that combines streams, but I don't see a good reason for
it. It has default settings, so if network output is slow might
accidentally store up to 1 second of pending data, increasing
latency.

Remove it in favour of doing any necessary buffering before
webrtcbin. If it turns out that there is a reason for it to
exist, the limits should probably be configurable and small.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3437>
subprojects/gst-plugins-bad/ext/webrtc/gstwebrtcbin.c