Move Ownership of RtpModules to VideoSendStream.

webrtc/video/vie_receiver.h

Index: webrtc/video/vie_receiver.h
diff --git a/webrtc/video/vie_receiver.h b/webrtc/video/vie_receiver.h
index 999e66d68b1e0006610df87e852fa407cc7227d6..c96bf5e107328ac6e8798ed03297d25d628dd61e 100644
--- a/webrtc/video/vie_receiver.h
+++ b/webrtc/video/vie_receiver.h
@@ -60,7 +60,7 @@ class ViEReceiver : public RtpData {
   uint32_t GetRemoteSsrc() const;
   int GetCsrcs(uint32_t* csrcs) const;
 
-  void Init(const std::vector<RtpRtcp*>& modules);
+  void Init(RtpRtcp* rtp_rtcp);
 
   RtpReceiver* GetRtpReceiver() const;
 
@@ -102,7 +102,7 @@ class ViEReceiver : public RtpData {
   RemoteBitrateEstimator* const remote_bitrate_estimator_;
 
   // TODO(pbos): Make const and set on construction.
-  std::vector<RtpRtcp*> rtp_rtcp_;
+  RtpRtcp* rtp_rtcp_;  // Owned by ViEChannel

[mflodman, 2016/04/15 08:25:42]

Maybe resolve pbos' todo when at this, should be easy since Init(...) is called
from the ViEChannel ctor anyway.

[perkj_webrtc, 2016/04/15 08:36:42]

Its not that easy since there is a circular dependency in ViEChannel between
VieReceiver and the ctor of the RTP module. So since that is unrelated to this
cl I would like to leave as is.

 
   RemoteNtpTimeEstimator ntp_estimator_;
   RTPPayloadRegistry rtp_payload_registry_;