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

Unified Diff: webrtc/video/video_receive_stream.cc

Issue 1814763002: Cleanup of webrtc::VideoRenderer (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Use a more specific DEPS rule. 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 6cc27941900831b4b5340d13fa8c3b3d21c19103..cb2515bec1fd2d7a4b6fe76e38c0227908424e38 100644
--- a/webrtc/video/video_receive_stream.cc
+++ b/webrtc/video/video_receive_stream.cc
@@ -388,14 +388,8 @@ int VideoReceiveStream::RenderFrame(const uint32_t /*stream_id*/,
if (vie_sync_.GetStreamSyncOffsetInMs(video_frame, &sync_offset_ms))
stats_proxy_.OnSyncOffsetUpdated(sync_offset_ms);
- // TODO(pbos): Wire up config_.render->IsTextureSupported() and convert if not
- // supported. Or provide methods for converting a texture frame in
- // VideoFrame.
-
if (config_.renderer != nullptr)
- config_.renderer->RenderFrame(
- video_frame,
- video_frame.render_time_ms() - clock_->TimeInMilliseconds());
+ config_.renderer->OnFrame(video_frame);
stats_proxy_.OnRenderedFrame(video_frame);
« webrtc/DEPS ('K') | « webrtc/video/video_quality_test.cc ('k') | webrtc/video_renderer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698