zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Set default audio level header extension value to 127.

Browse Source 
Having the minimum value as the default makes more sense than maximum.

Bug: b/232103634
Change-Id: Ia6a97f7a2a47bb74ed3b3316d95a1c6d00e2c16b
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/274260
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Commit-Queue: Jakob Ivarsson‎ <jakobi@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#38021}
This commit is contained in:
Jakob Ivarsson
2022-09-06 16:43:06 +02:00
committed by WebRTC LUCI CQ
parent 903ba6d736
commit 7e7a23fea4
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
modules/rtp_rtcp/source/rtp_sender_audio.h
View File

@ -62,7 +62,7 @@ class RTPSenderAudio {
// Store the audio level in dBov for
// header-extension-for-audio-level-indication.
// Valid range is [0,100]. Actual value is negative.
// Valid range is [0,127]. Actual value is negative.
int32_t SetAudioLevel(uint8_t level_dbov);
// Send a DTMF tone using RFC 2833 (4733)
@ -105,7 +105,7 @@ class RTPSenderAudio {
// Audio level indication.
// (https://datatracker.ietf.org/doc/draft-lennox-avt-rtp-audio-level-exthdr/)
uint8_t audio_level_dbov_ RTC_GUARDED_BY(send_audio_mutex_) = 0;
uint8_t audio_level_dbov_ RTC_GUARDED_BY(send_audio_mutex_) = 127;
OneTimeEvent first_packet_sent_;
absl::optional<uint32_t> encoder_rtp_timestamp_frequency_