From 3858477d4b751f85cbbcc6aec3570eebe4bf8e55 Mon Sep 17 00:00:00 2001 From: nisse Date: Mon, 11 Apr 2016 04:38:32 -0700 Subject: [PATCH] FakeVideoTrackRenderer is a thin wrapper over FakeVideoRenderer, only registering itself on a VideoSourceInterface on construction and removing itself on destruction. Let it inherit FakeVideoRenderer, instead of proxying all methods. BUG=webrtc:5426 Review URL: https://codereview.webrtc.org/1828173002 Cr-Commit-Position: refs/heads/master@{#12313} --- webrtc/api/test/fakevideotrackrenderer.h | 18 +----------------- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/webrtc/api/test/fakevideotrackrenderer.h b/webrtc/api/test/fakevideotrackrenderer.h index 3bd3d0b000..0d9b2488ab 100644 --- a/webrtc/api/test/fakevideotrackrenderer.h +++ b/webrtc/api/test/fakevideotrackrenderer.h @@ -16,8 +16,7 @@ namespace webrtc { -class FakeVideoTrackRenderer - : public rtc::VideoSinkInterface { +class FakeVideoTrackRenderer : public cricket::FakeVideoRenderer { public: FakeVideoTrackRenderer(VideoTrackInterface* video_track) : video_track_(video_track) { @@ -25,22 +24,7 @@ class FakeVideoTrackRenderer } ~FakeVideoTrackRenderer() { video_track_->RemoveSink(this); } - virtual void OnFrame(const cricket::VideoFrame& video_frame) override { - fake_renderer_.OnFrame(video_frame); - } - - int errors() const { return fake_renderer_.errors(); } - int width() const { return fake_renderer_.width(); } - int height() const { return fake_renderer_.height(); } - webrtc::VideoRotation rotation() const { return fake_renderer_.rotation(); } - bool black_frame() const { return fake_renderer_.black_frame(); } - - int num_rendered_frames() const { - return fake_renderer_.num_rendered_frames(); - } - private: - cricket::FakeVideoRenderer fake_renderer_; rtc::scoped_refptr video_track_; };