Move RtcEventLog object from inside VoiceEngine to Call.

webrtc/modules/rtp_rtcp/include/rtp_rtcp.h

Index: webrtc/modules/rtp_rtcp/include/rtp_rtcp.h
diff --git a/webrtc/modules/rtp_rtcp/include/rtp_rtcp.h b/webrtc/modules/rtp_rtcp/include/rtp_rtcp.h
index e0ace4c775d76c9df39d0d9a69954cb9c8b50df2..22d045ea182f753417d36ca1e036f24c8ba1b513 100644
--- a/webrtc/modules/rtp_rtcp/include/rtp_rtcp.h
+++ b/webrtc/modules/rtp_rtcp/include/rtp_rtcp.h
@@ -75,7 +75,6 @@ class RtpRtcp : public Module {
     BitrateStatisticsObserver* send_bitrate_observer;
     FrameCountObserver* send_frame_count_observer;
     SendSideDelayObserver* send_side_delay_observer;
-    RtcEventLog* event_log;

[stefan-webrtc, 2016/03/03 10:08:25]

Why do we have to make this change? If it really is needed to change this after
construction, then I'd prefer to set a proxy here instead and change the object
in the proxy instead.

[the sun, 2016/03/03 10:17:15]

That's a good idea! Put the proxy among the others in voice_engine/channel.cc;
we still need to be able to set the event log on voe::Channel after
construction.

[ivoc, 2016/03/10 13:15:36]

The reason for this change is that we no longer have the RtcEventLog* available
at the place where the VoE Channel is constructed (which is where the RtpRtcp
class is constructed) in the new setup. When constructing a ChannelProxy inside
the AudioReceiveStream, it doesn't actually construct a VoE Channel, but rather
finds the one with the correct channel id. 
I will implement a proxy for the event log like you suggest.

 
     RTC_DISALLOW_COPY_AND_ASSIGN(Configuration);
   };
@@ -649,6 +648,11 @@ class RtpRtcp : public Module {
     *   return -1 on failure else 0
     */
     virtual int32_t RequestKeyFrame() = 0;
+
+    /*
+     *  Set the RtcEventLog object.
+     */
+    virtual void SetRtcEventLog(webrtc::RtcEventLog* event_log) = 0;
 };
 }  // namespace webrtc
 #endif  // WEBRTC_MODULES_RTP_RTCP_INCLUDE_RTP_RTCP_H_