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

Unified Diff: webrtc/video/replay.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/replay.cc
diff --git a/webrtc/video/replay.cc b/webrtc/video/replay.cc
index 52b6ff6075b23dbd1ca7a62be6f44a065de1cf0b..3bfa7ff94975c6326259b93b0a88c2d15d5f6d1c 100644
--- a/webrtc/video/replay.cc
+++ b/webrtc/video/replay.cc
@@ -152,10 +152,9 @@ class FileRenderPassthrough : public VideoRenderer {
}
private:
- void RenderFrame(const VideoFrame& video_frame,
- int time_to_render_ms) override {
+ void OnFrame(const VideoFrame& video_frame) override {
if (renderer_ != nullptr)
- renderer_->RenderFrame(video_frame, time_to_render_ms);
+ renderer_->OnFrame(video_frame);
if (basename_.empty())
return;
if (last_width_ != video_frame.width() ||
@@ -182,8 +181,6 @@ class FileRenderPassthrough : public VideoRenderer {
PrintVideoFrame(video_frame, file_);
}
- bool IsTextureSupported() const override { return false; }
-
const std::string basename_;
VideoRenderer* const renderer_;
FILE* file_;

Powered by Google App Engine
This is Rietveld 408576698