rtptwcc: move TWCC-logic over to the TWCC-manager
authorHavard Graff <havard.graff@gmail.com>
Tue, 23 Feb 2021 08:44:05 +0000 (09:44 +0100)
committerGStreamer Marge Bot <gitlab-merge-bot@gstreamer-foundation.org>
Wed, 25 Aug 2021 08:36:06 +0000 (08:36 +0000)
commitb66c6714fae23651d757105080210e9837fa23d8
tree566d11c0dc92fe5e6ee29d4ab86da991cea33860
parentee361bf958d0172db0aa0ebe9b54fe9894cd5f30
rtptwcc: move TWCC-logic over to the TWCC-manager

Prevent cluttering up the rtpsession, and keeping things localized.

Also write TWCC-seqnums for *all* streams in the session if configured by
caps.

A while back WebRTC was not doing TWCC for audio, basically breaking the
whole idea of a "transport-wide seqnuencenumber" applying for all bundled
streams. However, they have since fixed this, and now it no longers
makes sense to be able to single out certain payloadtypes for
use with TWCC, rather just including them all.

This also makes using RTX, RED, FEC etc much simpler, as it will apply
to them all as they enter the rtpsession.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-good/-/merge_requests/927>
gst/rtpmanager/rtpsession.c
gst/rtpmanager/rtptwcc.c
gst/rtpmanager/rtptwcc.h