Negate flag name for prerender smoothing and update comments.
Further, strictly require VideoReceiveStream::Config::rendererer
to be non-null when the VideoReceiveStream is started. This is
already true by construction in the production code.
Bug: None
Change-Id: Ia0a41cfafa44215efc195a9eb6204194930c3dde
Reviewed-on: https://webrtc-review.googlesource.com/c/115040
Commit-Queue: Rasmus Brandt <brandtr@webrtc.org>
Reviewed-by: Niels Moller <nisse@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26384}diff --git a/call/video_receive_stream.h b/call/video_receive_stream.h
index b469553..3a55917 100644
--- a/call/video_receive_stream.h
+++ b/call/video_receive_stream.h
@@ -193,17 +193,16 @@
MediaTransportInterface* media_transport = nullptr;
- // Must not be 'nullptr' when the stream is started.
+ // Must always be set.
rtc::VideoSinkInterface<VideoFrame>* renderer = nullptr;
// Expected delay needed by the renderer, i.e. the frame will be delivered
// this many milliseconds, if possible, earlier than the ideal render time.
- // Only valid if 'renderer' is set.
int render_delay_ms = 10;
- // If set, pass frames on to the renderer as soon as they are
+ // If false, pass frames on to the renderer as soon as they are
// available.
- bool disable_prerenderer_smoothing = false;
+ bool enable_prerenderer_smoothing = true;
// Identifier for an A/V synchronization group. Empty string to disable.
// TODO(pbos): Synchronize streams in a sync group, not just video streams