From 5f935ef22e118830c9fd29d0244a25e1c32af31e Mon Sep 17 00:00:00 2001 From: kimvde Date: Tue, 8 Oct 2024 06:23:01 -0700 Subject: [PATCH] DefaultVideoSink: implement initialize() and isInitialized() The implementation is straightforward as there is nothing to init. PiperOrigin-RevId: 683596706 --- .../androidx/media3/exoplayer/video/DefaultVideoSink.java | 4 ++-- .../media3/exoplayer/video/PlaybackVideoGraphWrapper.java | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/DefaultVideoSink.java b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/DefaultVideoSink.java index ba176111b1..52e835602c 100644 --- a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/DefaultVideoSink.java +++ b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/DefaultVideoSink.java @@ -67,12 +67,12 @@ import java.util.concurrent.Executor; @Override public void initialize(Format sourceFormat) { - throw new UnsupportedOperationException(); + // Do nothing as there is no initialization needed. } @Override public boolean isInitialized() { - throw new UnsupportedOperationException(); + return true; } @Override diff --git a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java index 7648b96956..b7e4120c62 100644 --- a/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java +++ b/libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java @@ -396,6 +396,7 @@ public final class PlaybackVideoGraphWrapper implements VideoSinkProvider, Video } catch (VideoFrameProcessingException e) { throw new VideoSink.VideoSinkException(e, sourceFormat); } + defaultVideoSink.initialize(sourceFormat); state = STATE_INITIALIZED; return videoGraph.getProcessor(/* inputIndex= */ 0); }