Correcting the check for the return code produced by
AudioProcessing::ProcessReverseStream().
Before the change, only -1 was considered to be an error.
Allthough the error code scheme for AudioProcessing definitely
could be discussed, the current scheme have many error codes that
differ from -1 and thus were not caught by the old code.
BUG=webrtc:5237
Review URL: https://codereview.webrtc.org/1515073004
Cr-Original-Commit-Position: refs/heads/master@{#11003}
Cr-Mirrored-From: https://chromium.googlesource.com/external/webrtc
Cr-Mirrored-Commit: e5ae6f82374cb064d39cffffdb650840f734880f
diff --git a/voice_engine/output_mixer.cc b/voice_engine/output_mixer.cc
index 1b4d2e2..d46ee62 100644
--- a/voice_engine/output_mixer.cc
+++ b/voice_engine/output_mixer.cc
@@ -550,7 +550,7 @@
// ----------------------------------------------------------------------------
void OutputMixer::APMProcessReverseStream() {
- if (_audioProcessingModulePtr->ProcessReverseStream(&_audioFrame) == -1) {
+ if (_audioProcessingModulePtr->ProcessReverseStream(&_audioFrame) != 0) {
WEBRTC_TRACE(kTraceError, kTraceVoice, VoEId(_instanceId, -1),
"AudioProcessingModule::ProcessReverseStream() => error");
}