Injectable audio encoders: voice_engine/channel changes.

webrtc/voice_engine/channel_proxy.cc

 /*
  *  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 10 matching lines @@
 ChannelProxy::ChannelProxy() : channel_owner_(nullptr) {}
 
 ChannelProxy::ChannelProxy(const ChannelOwner& channel_owner) :
     channel_owner_(channel_owner) {
   RTC_CHECK(channel_owner_.channel());
   module_process_thread_checker_.DetachFromThread();
 }
 
 ChannelProxy::~ChannelProxy() {}
 
+bool ChannelProxy::SetEncoder(int payload_type,
+                              std::unique_ptr<AudioEncoder> encoder) {

[the sun, 2017/03/02 20:44:22]

add thread check + dcheck assumptions on pt range

[ossu, 2017/03/04 01:38:52]

I've added the thread check here (since ChannelProxy is where all other thread
checks are) and the payload type dchecks in Channel (since that is where the
actual implementation resides).

+  return channel()->SetEncoder(payload_type, std::move(encoder));
+}
+
 void ChannelProxy::SetRTCPStatus(bool enable) {
   RTC_DCHECK(worker_thread_checker_.CalledOnValidThread());
   channel()->SetRTCPStatus(enable);
 }
 
 void ChannelProxy::SetLocalSSRC(uint32_t ssrc) {
   RTC_DCHECK(worker_thread_checker_.CalledOnValidThread());
   int error = channel()->SetLocalSSRC(ssrc);
   RTC_DCHECK_EQ(0, error);
 }
@@ skipping 321 matching lines @@
   return channel()->SetSendCNPayloadType(type, frequency) == 0;
 }
 
 Channel* ChannelProxy::channel() const {
   RTC_DCHECK(channel_owner_.channel());
   return channel_owner_.channel();
 }
 
 }  // namespace voe
 }  // namespace webrtc