sjs/media
1
0
Fork 0
mirror of https://github.com/samsonjs/media.git synced 2026-04-27 15:07:40 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix not setting videoEffect in CompositingVSP

Browse source 
The current code only set the videoEffects when CVSP is initialized, which
happens after `player.prepare()`. But it's valid that videoEffects are set
before calling `prepare()`.

PiperOrigin-RevId: 566941216
This commit is contained in:
claincly 2023-09-20 06:09:20 -07:00 committed by Copybara-Service
parent 234f7525f0
commit 67900c3e81
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/CompositingVideoSinkProvider.java
View file

@ -88,12 +88,14 @@ import org.checkerframework.checker.nullness.qual.MonotonicNonNull;
try { try {
videoSinkImpl = videoSinkImpl =
new VideoSinkImpl(context, videoFrameProcessorFactory, renderControl, sourceFormat); new VideoSinkImpl(context, videoFrameProcessorFactory, renderControl, sourceFormat);
if (videoFrameMetadataListener != null) {
videoSinkImpl.setVideoFrameMetadataListener(videoFrameMetadataListener);
}
} catch (VideoFrameProcessingException e) { } catch (VideoFrameProcessingException e) {
throw new VideoSink.VideoSinkException(e, sourceFormat); throw new VideoSink.VideoSinkException(e, sourceFormat);
} }
if (videoFrameMetadataListener != null) {
videoSinkImpl.setVideoFrameMetadataListener(videoFrameMetadataListener);
}
videoSinkImpl.setVideoEffects(checkNotNull(videoEffects));
} }
/** Returns whether this provider is initialized for frame processing. */ /** Returns whether this provider is initialized for frame processing. */