Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(84)

Unified Diff: webrtc/video/video_receive_stream.cc

Issue 1813173002: Deletes the class VideoRendererCallback. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Break long lines. Add explicit keyword on IncomingVideoStream constructor. Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/video/video_receive_stream.cc
diff --git a/webrtc/video/video_receive_stream.cc b/webrtc/video/video_receive_stream.cc
index cb2515bec1fd2d7a4b6fe76e38c0227908424e38..b35322a1d658fbab8fdf6198f9b06bd89f409081 100644
--- a/webrtc/video/video_receive_stream.cc
+++ b/webrtc/video/video_receive_stream.cc
@@ -165,7 +165,6 @@ VideoReceiveStream::VideoReceiveStream(
this,
this)),
incoming_video_stream_(
- 0,
config.renderer ? config.renderer->SmoothsRenderedFrames() : false),
stats_proxy_(config_, clock_),
vie_channel_(&transport_adapter_,
@@ -382,8 +381,7 @@ void VideoReceiveStream::FrameCallback(VideoFrame* video_frame) {
}
}
-int VideoReceiveStream::RenderFrame(const uint32_t /*stream_id*/,
- const VideoFrame& video_frame) {
+void VideoReceiveStream::OnFrame(const VideoFrame& video_frame) {
int64_t sync_offset_ms;
if (vie_sync_.GetStreamSyncOffsetInMs(video_frame, &sync_offset_ms))
stats_proxy_.OnSyncOffsetUpdated(sync_offset_ms);
@@ -392,8 +390,6 @@ int VideoReceiveStream::RenderFrame(const uint32_t /*stream_id*/,
config_.renderer->OnFrame(video_frame);
stats_proxy_.OnRenderedFrame(video_frame);
-
- return 0;
}
// TODO(asapersson): Consider moving callback from video_encoder.h or

Powered by Google App Engine
This is Rietveld 408576698