Removed the indirect error message reporting in aec and aecm.

webrtc/modules/audio_processing/processing_component.cc

Index: webrtc/modules/audio_processing/processing_component.cc
diff --git a/webrtc/modules/audio_processing/processing_component.cc b/webrtc/modules/audio_processing/processing_component.cc
index 9e16d7c4eea6c14ff4c12e7c28f4f080b2e9ba1e..23ee6332d140d7269f041d1339cc0e5aa0eaa448 100644
--- a/webrtc/modules/audio_processing/processing_component.cc
+++ b/webrtc/modules/audio_processing/processing_component.cc
@@ -85,7 +85,7 @@ int ProcessingComponent::Initialize() {
 
     int err = InitializeHandle(handles_[i]);
     if (err != AudioProcessing::kNoError) {
-      return GetHandleError(handles_[i]);
+      return AudioProcessing::kUnspecifiedError;

[Andrew MacDonald, 2015/10/16 06:05:09]

I think it would probably be fine to remove all component-specific error
reporting, but I didn't understand from your explanation why you're only
removing these ones.

[peah-webrtc, 2015/10/16 08:14:39]

Done.

[peah-webrtc, 2015/10/16 08:14:39]

You are correct, this change is not really needed at this point, as the
GetHandleError in echo_cancellation_impl_ has been properly changed to handle
it. I'll revert this.

     }
   }
 
@@ -102,7 +102,7 @@ int ProcessingComponent::Configure() {
   for (int i = 0; i < num_handles_; i++) {
     int err = ConfigureHandle(handles_[i]);
     if (err != AudioProcessing::kNoError) {
-      return GetHandleError(handles_[i]);
+      return AudioProcessing::kUnspecifiedError;
     }
   }