summaryrefslogtreecommitdiff
path: root/ext/webrtc
diff options
context:
space:
mode:
authorSebastian Dröge <sebastian@centricular.com>2020-10-08 18:50:12 +0300
committerSebastian Dröge <sebastian@centricular.com>2020-10-08 18:53:12 +0300
commitcc7e98816f38fdc06a044de44e29d885dac1b8fa (patch)
tree977a18c338609dc08dfd1f40f1242be1e5da9542 /ext/webrtc
parenta40d6f49942eef1c3ab5c1e8a896849973152ff2 (diff)
downloadgstreamer-plugins-bad-cc7e98816f38fdc06a044de44e29d885dac1b8fa.tar.gz
Revert "webrtc: Save the media kind in the transceiver"
This reverts commit f54d8e99457996303b8477b1f3a710f0fabd1cc6. It breaks the CI until the C# bindings are fixed.
Diffstat (limited to 'ext/webrtc')
-rw-r--r--ext/webrtc/gstwebrtcbin.c69
1 files changed, 1 insertions, 68 deletions
diff --git a/ext/webrtc/gstwebrtcbin.c b/ext/webrtc/gstwebrtcbin.c
index 75d02a497..67d1d55e7 100644
--- a/ext/webrtc/gstwebrtcbin.c
+++ b/ext/webrtc/gstwebrtcbin.c
@@ -2870,47 +2870,6 @@ _media_add_rtx (GstSDPMedia * media, WebRTCTransceiver * trans,
}
}
-static GstWebRTCKind
-_kind_from_caps (const GstCaps * caps)
-{
- GstStructure *s;
- const gchar *media;
-
- if (gst_caps_get_size (caps) == 0)
- return GST_WEBRTC_KIND_UNKNOWN;
-
- s = gst_caps_get_structure (caps, 0);
-
- media = gst_structure_get_string (s, "media");
- if (media == NULL)
- return GST_WEBRTC_KIND_UNKNOWN;
-
- if (!g_strcmp0 (media, "audio"))
- return GST_WEBRTC_KIND_AUDIO;
-
- if (!g_strcmp0 (media, "video"))
- return GST_WEBRTC_KIND_VIDEO;
-
- return GST_WEBRTC_KIND_UNKNOWN;
-}
-
-static gboolean
-_update_transceiver_kind_from_caps (GstWebRTCRTPTransceiver * trans,
- const GstCaps * caps)
-{
- GstWebRTCKind kind = _kind_from_caps (caps);
-
- if (trans->kind == kind)
- return TRUE;
-
- if (trans->kind == GST_WEBRTC_KIND_UNKNOWN) {
- trans->kind = kind;
- return TRUE;
- } else {
- return FALSE;
- }
-}
-
static void
_get_rtx_target_pt_and_ssrc_from_caps (GstCaps * answer_caps, gint * target_pt,
guint * target_ssrc)
@@ -3221,10 +3180,6 @@ _create_answer_task (GstWebRTCBin * webrtc, const GstStructure * options,
} else {
trans = WEBRTC_TRANSCEIVER (rtp_trans);
}
- if (!_update_transceiver_kind_from_caps (rtp_trans, answer_caps))
- GST_WARNING_OBJECT (webrtc,
- "Trying to change transceiver %d kind from %d to %d",
- rtp_trans->mline, rtp_trans->kind, _kind_from_caps (answer_caps));
if (!trans->do_nack) {
answer_caps = gst_caps_make_writable (answer_caps);
@@ -3844,20 +3799,6 @@ _update_transceiver_from_sdp_media (GstWebRTCBin * webrtc,
rtp_trans->mline = media_idx;
- if (!g_strcmp0 (gst_sdp_media_get_media (media), "audio")) {
- if (rtp_trans->kind == GST_WEBRTC_KIND_VIDEO)
- GST_FIXME_OBJECT (webrtc,
- "Updating video transceiver to audio, which isn't fully supported.");
- rtp_trans->kind = GST_WEBRTC_KIND_AUDIO;
- }
-
- if (!g_strcmp0 (gst_sdp_media_get_media (media), "video")) {
- if (rtp_trans->kind == GST_WEBRTC_KIND_AUDIO)
- GST_FIXME_OBJECT (webrtc,
- "Updating audio transceiver to video, which isn't fully supported.");
- rtp_trans->kind = GST_WEBRTC_KIND_VIDEO;
- }
-
for (i = 0; i < gst_sdp_media_attributes_len (media); i++) {
const GstSDPAttribute *attr = gst_sdp_media_get_attribute (media, i);
@@ -5119,10 +5060,8 @@ gst_webrtc_bin_add_transceiver (GstWebRTCBin * webrtc,
"Created new unassociated transceiver %" GST_PTR_FORMAT, trans);
rtp_trans = GST_WEBRTC_RTP_TRANSCEIVER (trans);
- if (caps) {
+ if (caps)
rtp_trans->codec_preferences = gst_caps_ref (caps);
- _update_transceiver_kind_from_caps (rtp_trans, caps);
- }
return gst_object_ref (trans);
}
@@ -5973,12 +5912,6 @@ gst_webrtc_bin_request_new_pad (GstElement * element, GstPadTemplate * templ,
}
pad->trans = gst_object_ref (trans);
- if (caps && name && !_update_transceiver_kind_from_caps (trans, caps))
- GST_WARNING_OBJECT (webrtc,
- "Trying to create pad %s with caps %" GST_PTR_FORMAT
- " but transceiver %d already exists with a different"
- " media type", name, caps, serial);
-
pad->block_id = gst_pad_add_probe (GST_PAD (pad), GST_PAD_PROBE_TYPE_BLOCK |
GST_PAD_PROBE_TYPE_BUFFER | GST_PAD_PROBE_TYPE_BUFFER_LIST,
(GstPadProbeCallback) sink_pad_block, NULL, NULL);