WebRtcVoiceEngine: Use AudioDecoderFactory to generate recv codecs.

webrtc/media/engine/webrtcvoiceengine.h

Index: webrtc/media/engine/webrtcvoiceengine.h
diff --git a/webrtc/media/engine/webrtcvoiceengine.h b/webrtc/media/engine/webrtcvoiceengine.h
index 52983fad2dd8bae45e2de0f287e84d04a285c813..5bc313f12d2e4123cbf0c303bdd2941bfd26da2f 100644
--- a/webrtc/media/engine/webrtcvoiceengine.h
+++ b/webrtc/media/engine/webrtcvoiceengine.h
@@ -131,6 +131,8 @@ class WebRtcVoiceEngine final : public webrtc::TraceCallback  {
   int CreateVoEChannel();
   webrtc::AudioDeviceModule* adm();
 
+  AudioCodecs CollectRecvCodecs() const;

[ivoc, 2016/07/05 14:43:51]

I would prefer not to abbreviate "Recv" in the method name (unless this is
commonly done?).

[ossu, 2016/07/05 15:31:57]

It's done eeeeverywhere, except where it isn't. :)
We really should move for uniformity at some point. In this part of the code,
it's common with recv, though, and it does update recv_codecs_, which makes its
name apt.

+
   rtc::ThreadChecker signal_thread_checker_;
   rtc::ThreadChecker worker_thread_checker_;
 
@@ -140,7 +142,8 @@ class WebRtcVoiceEngine final : public webrtc::TraceCallback  {
   // The primary instance of WebRtc VoiceEngine.
   std::unique_ptr<VoEWrapper> voe_wrapper_;
   rtc::scoped_refptr<webrtc::AudioState> audio_state_;
-  std::vector<AudioCodec> codecs_;
+  std::vector<AudioCodec> send_codecs_;
+  std::vector<AudioCodec> recv_codecs_;
   std::vector<WebRtcVoiceMediaChannel*> channels_;
   webrtc::Config voe_config_;
   bool is_dumping_aec_ = false;