CodecOwner::SetEncoders: Return error code when given bad arguments

webrtc/modules/audio_coding/main/acm2/codec_owner.h

 /*
  *  Copyright (c) 2015 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 11 matching lines @@
 
 class AudioDecoder;
 
 namespace acm2 {
 
 class CodecOwner {
  public:
   CodecOwner();
   ~CodecOwner();
 
-  void SetEncoders(const CodecInst& speech_inst,
+  // Start using the specified encoder. Returns false on error.
+  // TODO(kwiberg): Don't handle errors here (bug 5033)
+  bool SetEncoders(const CodecInst& speech_inst,
                    int cng_payload_type,
                    ACMVADMode vad_mode,
-                   int red_payload_type);
+                   int red_payload_type) WARN_UNUSED_RESULT;
 
   void SetEncoders(AudioEncoder* external_speech_encoder,
                    int cng_payload_type,
                    ACMVADMode vad_mode,
                    int red_payload_type);
 
   void ChangeCngAndRed(int cng_payload_type,
                        ACMVADMode vad_mode,
                        int red_payload_type);
 
@@ skipping 23 matching lines @@
   // are active.
   rtc::scoped_ptr<AudioEncoder> cng_encoder_;
   rtc::scoped_ptr<AudioEncoder> red_encoder_;
 
   RTC_DISALLOW_COPY_AND_ASSIGN(CodecOwner);
 };
 
 }  // namespace acm2
 }  // namespace webrtc
 #endif  // WEBRTC_MODULES_AUDIO_CODING_MAIN_ACM2_CODEC_OWNER_H_