Move Ownership of RtpModules to VideoSendStream.

webrtc/video/payload_router.cc

Index: webrtc/video/payload_router.cc
diff --git a/webrtc/video/payload_router.cc b/webrtc/video/payload_router.cc
index 968d82df62a0fa2c7153f49ca70202ea6c05090a..8cd1ed30783049b7a1d34c2dc3b45b73b4835a57 100644
--- a/webrtc/video/payload_router.cc
+++ b/webrtc/video/payload_router.cc
@@ -16,8 +16,10 @@
 
 namespace webrtc {
 
-PayloadRouter::PayloadRouter()
-    : active_(false), num_sending_modules_(0) {}
+PayloadRouter::PayloadRouter(const std::vector<RtpRtcp*>& rtp_modules)
+    : active_(false), num_sending_modules_(0), rtp_modules_(rtp_modules) {
+  SetSendingRtpModules(1);

[stefan-webrtc, 2016/04/12 07:59:10]

Is it necessary to take the lock in SetSendingRtpModules here? Maybe we can have
a version of the method which doesn't take the lock.

[perkj_webrtc, 2016/04/13 11:43:49]

no, its not since it is the ctor. done

+}
 
 PayloadRouter::~PayloadRouter() {}
 
@@ -26,12 +28,6 @@ size_t PayloadRouter::DefaultMaxPayloadLength() {
   return IP_PACKET_SIZE - kIpUdpSrtpLength;
 }
 
-void PayloadRouter::Init(
-    const std::vector<RtpRtcp*>& rtp_modules) {
-  RTC_DCHECK(rtp_modules_.empty());
-  rtp_modules_ = rtp_modules;
-}
-
 void PayloadRouter::set_active(bool active) {
   rtc::CritScope lock(&crit_);
   if (active_ == active)