Remove frame time scheduing in IncomingVideoStream

webrtc/common_video/include/incoming_video_stream.h

Index: webrtc/common_video/include/incoming_video_stream.h
diff --git a/webrtc/common_video/include/incoming_video_stream.h b/webrtc/common_video/include/incoming_video_stream.h
index cd0d65385e886b5453bc28c752d693b7c29e145e..c963da0931e0056db81a0cb56dd22db9b9072d15 100644
--- a/webrtc/common_video/include/incoming_video_stream.h
+++ b/webrtc/common_video/include/incoming_video_stream.h
@@ -19,6 +19,7 @@ namespace webrtc {
 class CriticalSectionWrapper;
 class EventTimerWrapper;
 class ThreadWrapper;
+class VideoRenderer;
 
 class VideoRenderCallback {
  public:
@@ -32,6 +33,7 @@ class VideoRenderCallback {
 class IncomingVideoStream : public VideoRenderCallback {
  public:
   explicit IncomingVideoStream(uint32_t stream_id);

[pbos-webrtc, 2015/11/23 14:28:05]

Can you just have a bool disable_prerender_smoothing here? We should know this
on construction, so you should be able to pass
renderer->PrerenderSmoothingDisabled(), this shouldn't ever change during
execution. Then also only have one constructor.

[mflodman, 2015/11/23 14:43:44]

+1, I assume this won't change during the life time of the renderer. And then
only have one constructor.

[qiangchen, 2015/11/24 00:21:05]

Done.

But one concern is the compatibility break, if we remove a signature of
constructor.

[mflodman, 2015/11/24 12:21:17]

AFAIK this is only used by internal WebRTC code and should be fine as long as it
builds locally on on the bots.

+  IncomingVideoStream(uint32_t stream_id, VideoRenderer* renderer);
   ~IncomingVideoStream();
 
   // Get callback to deliver frames to the module.
@@ -73,6 +75,7 @@ class IncomingVideoStream : public VideoRenderCallback {
   enum { kFrameRatePeriodMs = 1000 };
 
   uint32_t const stream_id_;
+  VideoRenderer* const renderer_;
   // Critsects in allowed to enter order.
   const rtc::scoped_ptr<CriticalSectionWrapper> stream_critsect_;
   const rtc::scoped_ptr<CriticalSectionWrapper> thread_critsect_;