Clean up of CongestionController.

webrtc/video/video_receive_stream.h

Index: webrtc/video/video_receive_stream.h
diff --git a/webrtc/video/video_receive_stream.h b/webrtc/video/video_receive_stream.h
index 9cb6d40e50209beff5c71530dec4299bcd7237b9..71fe173eb813072df097464cec0d7d53d6229f11 100644
--- a/webrtc/video/video_receive_stream.h
+++ b/webrtc/video/video_receive_stream.h
@@ -33,6 +33,7 @@ namespace webrtc {
 class CallStats;
 class CongestionController;
 class VoiceEngine;
+class VieRemb;
 
 namespace internal {
 
@@ -46,7 +47,8 @@ class VideoReceiveStream : public webrtc::VideoReceiveStream,
                      const VideoReceiveStream::Config& config,
                      webrtc::VoiceEngine* voice_engine,
                      ProcessThread* process_thread,
-                     CallStats* call_stats);
+                     CallStats* call_stats,
+                     VieRemb* remb);

[the sun, 2016/02/08 10:39:23]

For the audio streams, I've created a shared AudioState object, which is passed
down. I think it could make sense to do something similar for video. Then we
could avoid passing naked pointers like this, instead pass the state as a ref
counted object and get the various fields with accessors.

See:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/webrtc...
and
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/webrtc...

I'm not suggesting you do it in this CL, but maybe think about it?

[stefan-webrtc, 2016/02/08 13:30:54]

Possibly. pbos, what's your opinion on this?

   ~VideoReceiveStream() override;
 
   // webrtc::ReceiveStream implementation.
@@ -86,6 +88,7 @@ class VideoReceiveStream : public webrtc::VideoReceiveStream,
 
   CongestionController* const congestion_controller_;
   CallStats* const call_stats_;
+  VieRemb* const remb_;
 
   rtc::scoped_ptr<VideoCodingModule> vcm_;
   IncomingVideoStream incoming_video_stream_;