From f6d0c51c045f67c8f97228fd58e76876180493b3 Mon Sep 17 00:00:00 2001 From: Marco Castelluccio Date: Sun, 1 Dec 2024 01:18:07 +0000 Subject: [PATCH] Bug 1933658 [wpt PR 49392] - Update stereo opus test to expect upmixing., a=testonly MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Automatic update from web-platform-tests Update stereo opus test to expect upmixing. Update the test that sends and receives a local mono track with opus that on the remote side the channelCount is 2 because the opus decoder performs upmixing. This strange behavior is option 1) discussed in https://github.com/w3c/webrtc-pc/issues/3010. This allows for some optimization and simplifications which aids in virtual audio SSRC use case of flip-flopping between mono and stereo signals on the wire and not having to re-initialize the decoder (which could cause glitches). The vibe from the November Virtual Interim (https://www.w3.org/2024/11/19-webrtc-minutes.html#561c) was to standardize what libwebrtc does when stereo is enabled, and this is what it does. Note: We still fail the test today though because "stereo opus" has not been enabled-by-default. When it is the expectation is that we pass both of these tests. Bug: webrtc:379996136 Change-Id: Ibd6ee91a59d7c673bf53ad2302ba33f16b8b4ede Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6049355 Reviewed-by: Harald Alvestrand Commit-Queue: Henrik Boström Cr-Commit-Position: refs/heads/main{#1388674} -- wpt-commits: d97f56c5831beb33c9302d2dd2707554bd842116 wpt-pr: 49392 UltraBlame original commit: de89aa35ef835eeb58e01f3e96b8c4de7bef4e4d --- ...ingParameters-codec-opus-stereo.https.html | 33 +++++++++++++++++-- 1 file changed, 31 insertions(+), 2 deletions(-) diff --git a/testing/web-platform/tests/webrtc/RTCRtpEncodingParameters-codec-opus-stereo.https.html b/testing/web-platform/tests/webrtc/RTCRtpEncodingParameters-codec-opus-stereo.https.html index 47da849fbd878..d7a378f3ca506 100644 --- a/testing/web-platform/tests/webrtc/RTCRtpEncodingParameters-codec-opus-stereo.https.html +++ b/testing/web-platform/tests/webrtc/RTCRtpEncodingParameters-codec-opus-stereo.https.html @@ -562,6 +562,24 @@ remoteTrack ) ; +/ +/ +The +stereo +opus +decoder +outputs +stereo +regardless +of +the +signal +on +the +/ +/ +wire +. assert_equals ( await @@ -570,13 +588,19 @@ t remoteTrack ) -1 +2 ' Remote track is -also +stereo +( +despite +local +track +being mono +) ' ) ; @@ -590,6 +614,11 @@ track with opus +( +stereo +decoder +upmix +) ' ) ;