Early initialize recording on the ADM from WebRtcVoiceMediaChannel.

webrtc/voice_engine/channel_proxy.cc

Index: webrtc/voice_engine/channel_proxy.cc
diff --git a/webrtc/voice_engine/channel_proxy.cc b/webrtc/voice_engine/channel_proxy.cc
index 10c88212024aa2ea911c09ccb97bcb2fd15ca65e..d577fb4ea2df8dd2f30859893e1409ef23b55d59 100644
--- a/webrtc/voice_engine/channel_proxy.cc
+++ b/webrtc/voice_engine/channel_proxy.cc
@@ -158,6 +158,12 @@ void ChannelProxy::SetSink(std::unique_ptr<AudioSinkInterface> sink) {
   channel()->SetSink(std::move(sink));
 }
 
+void ChannelProxy::StartSend() {
+  RTC_DCHECK(thread_checker_.CalledOnValidThread());
+  int32_t error = channel()->StartSend();
+  RTC_DCHECK_EQ(0, error);

[Taylor Brandstetter, 2016/03/29 01:05:38]

Is there a reason to switch from logging an error to using RTC_DCHECK_EQ?

[the sun, 2016/03/31 10:49:47]

Yes, there is a reason. AFAICT StartSend() can never fail, so this isn't
something we need to log. Rather, it is a programming logic error, i.e. either I
read the code wrong just now, or someone changes the code in the below layers so
the assumption doesn't hold anymore. In both cases a logic error, therefore an
assertion is appropriate.

[Taylor Brandstetter, 2016/03/31 18:11:26]

If it should never fail, it seems like it would make sense to remove the return
code in order to make that requirement explicit. But that's a separate matter
from this CL, so this is good for now.

[the sun, 2016/04/01 09:07:46]

voe::Channel is still accessed via the legacy VoE interfaces (VoEBase, etc),
which are being deprecated. I don't see much point in changing code in them.
Instead, I've introduced the interim ChannelProxy class which provides a
cleaned-up "view" of voe::Channel to the new AudioReceive/SendStream classes, as
well as allowing to mock voe::Channel functionality so the stream classes can be
tested. Once the legacy APIs are removed, voe::Channel can more easily be
changed, and its functionality assimilated by the stream classes.

+}
+
 Channel* ChannelProxy::channel() const {
   RTC_DCHECK(channel_owner_.channel());
   return channel_owner_.channel();