From 998a9f94041945683b4aad40c94ccef447bee592 Mon Sep 17 00:00:00 2001 From: Raja Subramanian Date: Thu, 28 Aug 2025 14:47:57 +0530 Subject: [PATCH] Switch known rids from 012 -> 210, used by OBS. (#3882) * Switch known rids from 012 -> 210, used by OBS. * static check --- pkg/sfu/buffer/videolayerutils.go | 10 +++++----- pkg/sfu/buffer/videolayerutils_test.go | 16 ++++++++-------- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/pkg/sfu/buffer/videolayerutils.go b/pkg/sfu/buffer/videolayerutils.go index e4b923744..5045c6d08 100644 --- a/pkg/sfu/buffer/videolayerutils.go +++ b/pkg/sfu/buffer/videolayerutils.go @@ -27,16 +27,16 @@ const ( halfResolutionH = "h" fullResolutionF = "f" - quarterResolution0 = "0" + quarterResolution2 = "2" halfResolution1 = "1" - fullResolution2 = "2" + fullResolution0 = "0" ) type VideoLayersRid [DefaultMaxLayerSpatial + 1]string var ( videoLayersRidQHF = VideoLayersRid{quarterResolutionQ, halfResolutionH, fullResolutionF} - videoLayersRid012 = VideoLayersRid{quarterResolution0, halfResolution1, fullResolution2} + videoLayersRid210 = VideoLayersRid{quarterResolution2, halfResolution1, fullResolution0} DefaultVideoLayersRid = videoLayersRidQHF ) @@ -511,8 +511,8 @@ func NormalizeVideoLayersRid(rids VideoLayersRid) VideoLayersRid { normalize(videoLayersRidQHF) } - if isVideoLayersRidKnown(rids, videoLayersRid012) { - normalize(videoLayersRid012) + if isVideoLayersRidKnown(rids, videoLayersRid210) { + normalize(videoLayersRid210) } return out diff --git a/pkg/sfu/buffer/videolayerutils_test.go b/pkg/sfu/buffer/videolayerutils_test.go index 357234e25..a7c5ad26d 100644 --- a/pkg/sfu/buffer/videolayerutils_test.go +++ b/pkg/sfu/buffer/videolayerutils_test.go @@ -867,19 +867,19 @@ func TestNormalizeVideoLayersRid(t *testing.T) { VideoLayersRid{"q", "h", ""}, }, { - "012", - videoLayersRid012, - videoLayersRid012, + "210", + videoLayersRid210, + videoLayersRid210, }, { - "scrambled 012", + "scrambled 210", VideoLayersRid{"2", "0", "1"}, - videoLayersRid012, + videoLayersRid210, }, { - "partial 012", - VideoLayersRid{"2", "1"}, - VideoLayersRid{"1", "2", ""}, + "partial 210", + VideoLayersRid{"1", "2"}, + VideoLayersRid{"2", "1", ""}, }, }