Removed the indirect error message reporting in aec and aecm.

webrtc/modules/audio_processing/processing_component.cc

 /*
  *  Copyright (c) 2011 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 67 matching lines @@
   for (int i = 0; i < num_handles_; i++) {
     if (handles_[i] == NULL) {
       handles_[i] = CreateHandle();
       if (handles_[i] == NULL) {
         return AudioProcessing::kCreationFailedError;
       }
     }
 
     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.

     }
   }
 
   initialized_ = true;
   return Configure();
 }
 
 int ProcessingComponent::Configure() {
   if (!initialized_) {
     return AudioProcessing::kNoError;
   }
 
   assert(static_cast<int>(handles_.size()) >= num_handles_);
   for (int i = 0; i < num_handles_; i++) {
     int err = ConfigureHandle(handles_[i]);
     if (err != AudioProcessing::kNoError) {
-      return GetHandleError(handles_[i]);
+      return AudioProcessing::kUnspecifiedError;
     }
   }
 
   return AudioProcessing::kNoError;
 }
 }  // namespace webrtc