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

Unified Diff: webrtc/modules/audio_conference_mixer/interface/audio_conference_mixer_defines.h

Issue 1311733003: Stylizing AudioConferenceMixer. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: refine two commenting Created 5 years, 4 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
Index: webrtc/modules/audio_conference_mixer/interface/audio_conference_mixer_defines.h
diff --git a/webrtc/modules/audio_conference_mixer/interface/audio_conference_mixer_defines.h b/webrtc/modules/audio_conference_mixer/interface/audio_conference_mixer_defines.h
index 784f63a6612d4f571a02b066dab7af2cd62d7701..d15b7fca02c0537298c8eda457d225d7dee10c39 100644
--- a/webrtc/modules/audio_conference_mixer/interface/audio_conference_mixer_defines.h
+++ b/webrtc/modules/audio_conference_mixer/interface/audio_conference_mixer_defines.h
@@ -25,14 +25,15 @@ public:
// audio every time it's called.
//
// If it returns -1, the frame will not be added to the mix.
- virtual int32_t GetAudioFrame(const int32_t id, AudioFrame& audioFrame) = 0;
+ virtual int32_t GetAudioFrame(int32_t id,
+ AudioFrame* audioFrame) = 0;
- // mixed will be set to true if the participant was mixed this mix iteration
- int32_t IsMixed(bool& mixed) const;
+ // Returns true if the participant was mixed this mix iteration.
+ bool IsMixed() const;
// This function specifies the sampling frequency needed for the AudioFrame
// for future GetAudioFrame(..) calls.
- virtual int32_t NeededFrequency(const int32_t id) = 0;
+ virtual int32_t NeededFrequency(int32_t id) const = 0;
MixHistory* _mixHistory;
protected:

Powered by Google App Engine
This is Rietveld 408576698