sjs/media
1
0
Fork 0
mirror of https://github.com/samsonjs/media.git synced 2026-03-26 09:35:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix not able to set a null output surface.

Browse source 
Previously, after calling MCVR.setOutput() with null, `frameProcessorManager`'s output surface is cleared. What was unexpected is `ExoPlayerInternal` notifies a zero output resolution after clearing the output surface. This zero resolution causes FrameProcessor to fail.

PiperOrigin-RevId: 501861993
This commit is contained in:
claincly 2023-01-13 17:02:58 +00:00 committed by Rohit Singh
parent d16e84e296
commit 492dfeb0c7
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/MediaCodecVideoRenderer.java
View file

@ -686,7 +686,10 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
break;
case MSG_SET_VIDEO_OUTPUT_RESOLUTION:
Size outputResolution = (Size) checkNotNull(message);
if (displaySurface != null && frameProcessorManager.isEnabled()) {
if (outputResolution.getWidth() != 0
&& outputResolution.getHeight() != 0
&& displaySurface != null
&& frameProcessorManager.isEnabled()) {
frameProcessorManager.setOutputSurfaceInfo(displaySurface, outputResolution);
}
break;
@ -2051,6 +2054,7 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
checkNotNull(frameProcessor).setOutputSurfaceInfo(null);
currentSurfaceAndSize = null;
}
/**
* Sets the input surface info.
*