Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1251)

Unified Diff: talk/media/base/fakemediaengine.h

Issue 1378513003: Change 'mute' parameter of MediaChannel::SetAudioSend()/SetVideoSend() to 'enable'. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: rebase Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « talk/app/webrtc/webrtcsession.cc ('k') | talk/media/base/mediachannel.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: talk/media/base/fakemediaengine.h
diff --git a/talk/media/base/fakemediaengine.h b/talk/media/base/fakemediaengine.h
index b53c645f75ef7ccfa26c5bf0a398ae8ecf2bee14..4e39a2a701adb2b8a16c5f09c1e62ac422cb37ed 100644
--- a/talk/media/base/fakemediaengine.h
+++ b/talk/media/base/fakemediaengine.h
@@ -275,16 +275,16 @@ class FakeVoiceMediaChannel : public RtpHelper<VoiceMediaChannel> {
}
return set_sending(flag != SEND_NOTHING);
}
- virtual bool SetAudioSend(uint32 ssrc, bool mute,
+ virtual bool SetAudioSend(uint32 ssrc, bool enable,
const AudioOptions* options,
AudioRenderer* renderer) {
if (!SetLocalRenderer(ssrc, renderer)) {
return false;
}
- if (!RtpHelper<VoiceMediaChannel>::MuteStream(ssrc, mute)) {
+ if (!RtpHelper<VoiceMediaChannel>::MuteStream(ssrc, !enable)) {
return false;
}
- if (!mute && options) {
+ if (enable && options) {
return SetOptions(*options);
}
return true;
@@ -550,12 +550,12 @@ class FakeVideoMediaChannel : public RtpHelper<VideoMediaChannel> {
}
virtual bool SetSend(bool send) { return set_sending(send); }
- virtual bool SetVideoSend(uint32 ssrc, bool mute,
+ virtual bool SetVideoSend(uint32 ssrc, bool enable,
const VideoOptions* options) {
- if (!RtpHelper<VideoMediaChannel>::MuteStream(ssrc, mute)) {
+ if (!RtpHelper<VideoMediaChannel>::MuteStream(ssrc, !enable)) {
return false;
}
- if (!mute && options) {
+ if (enable && options) {
return SetOptions(*options);
}
return true;
« no previous file with comments | « talk/app/webrtc/webrtcsession.cc ('k') | talk/media/base/mediachannel.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698