Move Ownership of RtpModules to VideoSendStream.

webrtc/video/video_send_stream.h

Index: webrtc/video/video_send_stream.h
diff --git a/webrtc/video/video_send_stream.h b/webrtc/video/video_send_stream.h
index 8b3d064f3e34988f352a7a15ec3fa75f2d9cd3c2..8fbc9f89e442e1aee10df5a274f4bfd606ec6e0d 100644
--- a/webrtc/video/video_send_stream.h
+++ b/webrtc/video/video_send_stream.h
@@ -42,7 +42,8 @@ namespace internal {
 
 class VideoSendStream : public webrtc::VideoSendStream,
                         public webrtc::CpuOveruseObserver,
-                        public webrtc::BitrateAllocatorObserver {
+                        public webrtc::BitrateAllocatorObserver,
+                        public webrtc::VCMProtectionCallback {
  public:
   VideoSendStream(int num_cpu_cores,
                   ProcessThread* module_process_thread,
@@ -81,6 +82,13 @@ class VideoSendStream : public webrtc::VideoSendStream,
                         uint8_t fraction_loss,
                         int64_t rtt) override;
 
+  // implements webrtc::VCMProtectionCallback.

[pbos-webrtc, 2016/04/07 16:02:08]

Implements

[perkj_webrtc, 2016/04/08 10:59:04]

Done.

+  int ProtectionRequest(const FecProtectionParams* delta_params,
+                        const FecProtectionParams* key_params,
+                        uint32_t* sent_video_rate_bps,
+                        uint32_t* sent_nack_rate_bps,
+                        uint32_t* sent_fec_rate_bps) override;
+
  private:
   static bool EncoderThreadFunction(void* obj);
   void EncoderProcess();
@@ -103,16 +111,13 @@ class VideoSendStream : public webrtc::VideoSendStream,
   volatile int stop_encoder_thread_;
 
   OveruseFrameDetector overuse_detector_;
-  PayloadRouter payload_router_;
   EncoderStateFeedback encoder_feedback_;
-  ViEChannel vie_channel_;
-  ViEReceiver* const vie_receiver_;
   ViEEncoder vie_encoder_;
   VideoCodingModule* const vcm_;
-  // TODO(pbos): Move RtpRtcp ownership to VideoSendStream.
-  // RtpRtcp modules, currently owned by ViEChannel but ownership should
-  // eventually move here.
+
+  // RtpRtcp modules, declared here as they  use other members on construction.

[pbos-webrtc, 2016/04/07 16:02:08]

remove double space.

[perkj_webrtc, 2016/04/08 10:59:04]

Done.

   const std::vector<RtpRtcp*> rtp_rtcp_modules_;
+  PayloadRouter payload_router_;
   VideoCaptureInput input_;
 };
 }  // namespace internal