commit e987f2b76567d067e1aeae23400978432c9d68c3
author Magnus Jedvert <magjed@webrtc.org> Mon Apr 23 16:14:47 2018 +0200
committer Commit Bot <commit-bot@chromium.org> Mon Apr 23 16:04:11 2018 +0000
tree a453b0ffe737fda7d00ed2e93e6d78e894a10159
parent b9ac121598e17af77260cdf86dc465afecd099b2

Android: Stop using VideoRenderer class

This CL updates the WebRTC code to stop using the old VideoRenderer and
VideoRenderer.I420Frame classes and instead use the new VideoSink and
VideoFrame classes.

This CL is the first step and the old classes are still left in the code
for now to keep backwards compatibility.

Bug: webrtc:9181
Change-Id: Ib0caa18cbaa2758b7859e850ddcaba003cfb06d6
Reviewed-on: https://webrtc-review.googlesource.com/71662
Reviewed-by: Sami Kalliomäki <sakal@webrtc.org>
Commit-Queue: Magnus Jedvert <magjed@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22989}

sdk/android/api/org/webrtc/VideoFileRenderer.java

diff --git a/sdk/android/api/org/webrtc/VideoFileRenderer.java b/sdk/android/api/org/webrtc/VideoFileRenderer.java
index 1b9fa48..c599491 100644
--- a/sdk/android/api/org/webrtc/VideoFileRenderer.java
+++ b/sdk/android/api/org/webrtc/VideoFileRenderer.java
@@ -25,7 +25,7 @@
  * Can be used to save the video frames to file.
  */
 @JNINamespace("webrtc::jni")
-public class VideoFileRenderer implements VideoRenderer.Callbacks, VideoSink {
+public class VideoFileRenderer implements VideoSink {
   private static final String TAG = "VideoFileRenderer";
 
   private final HandlerThread renderThread;
@@ -74,13 +74,6 @@
   }
 
   @Override
-  public void renderFrame(final VideoRenderer.I420Frame i420Frame) {
-    final VideoFrame frame = i420Frame.toVideoFrame();
-    onFrame(frame);
-    frame.release();
-  }
-
-  @Override
   public void onFrame(VideoFrame frame) {
     frame.retain();
     renderThreadHandler.post(() -> renderFrameOnRenderThread(frame));