-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'roxie/secondlife-fix-bars' of github.com:secondlife/3p-…
…webrtc-build into secondlife
- Loading branch information
Showing
2 changed files
with
41 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,24 @@ | ||
diff --git a/media/engine/webrtc_voice_engine.cc b/media/engine/webrtc_voice_engine.cc | ||
index 35e2657876..be6ed226f1 100644 | ||
--- a/media/engine/webrtc_voice_engine.cc | ||
+++ b/media/engine/webrtc_voice_engine.cc | ||
@@ -2280,6 +2280,19 @@ bool WebRtcVoiceMediaChannel::MuteStream(uint32_t ssrc, bool muted) { | ||
for (const auto& kv : send_streams_) { | ||
all_muted = all_muted && kv.second->muted(); | ||
} | ||
+ // Because multiple WebRTCVoiceMediaChannels can be | ||
+ // associated with a single AudioProcessing module, | ||
+ // muting when all streams of this media channel | ||
+ // are muted is incorrect, because streams on other | ||
+ // channels may not be muted. So, for now, | ||
+ // we'll simply not disable audio processing when | ||
+ // streams are muted. | ||
+ // | ||
+ // TODO(roxanneskelly) | ||
+ // Track which WebRTCVoiceMediaChannels are all_muted | ||
+ // and only call set_output_will_be_muted when all of them | ||
+ // are muted. | ||
+ all_muted = false; | ||
webrtc::AudioProcessing* ap = engine()->apm(); | ||
if (ap) { | ||
ap->set_output_will_be_muted(all_muted); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters