loliball/media
1
0
Fork 0
mirror of https://github.com/androidx/media.git synced 2025-04-29 14:26:50 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Share some code for setting video output

Browse Source 
The code for setting the video output is almost the same across both places,
with one callsite supporting less types of video output. I think it's still
better to share the code, to minimize the margin for mistake later.

PiperOrigin-RevId: 751423005
This commit is contained in:
claincly 2025-04-25 08:18:37 -07:00 committed by Copybara-Service
parent 8bf658cd79
commit 09ce64ec21
1 changed files with 25 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
libraries/transformer/src/main/java/androidx/media3/transformer/CompositionPlayer.java
View File

@ -400,6 +400,7 @@ public final class CompositionPlayer extends SimpleBasePlayer
* @param composition The {@link Composition} to play. Every {@link EditedMediaItem} in the {@link
* Composition} must have its {@link EditedMediaItem#durationUs} set.
*/
@SuppressWarnings("FutureReturnValueIgnored")
public void setComposition(Composition composition) {
verifyApplicationThread();
checkArgument(!composition.sequences.isEmpty());
@ -412,20 +413,9 @@ public final class CompositionPlayer extends SimpleBasePlayer
}
setCompositionInternal(composition);
if (videoOutput != null) {
if (videoOutput instanceof SurfaceHolder) {
setVideoSurfaceHolderInternal((SurfaceHolder) videoOutput);
} else if (videoOutput instanceof SurfaceView) {
SurfaceView surfaceView = (SurfaceView) videoOutput;
setVideoSurfaceHolderInternal(surfaceView.getHolder());
} else if (videoOutput instanceof Surface) {
setVideoSurfaceInternal((Surface) videoOutput, checkNotNull(videoOutputSize));
} else {
throw new IllegalStateException(videoOutput.getClass().toString());
}
}
// Update the composition field at the end after everything else has been set.
this.composition = composition;
maybeSetVideoOutput();
}
/**
@ -444,7 +434,6 @@ public final class CompositionPlayer extends SimpleBasePlayer
}
/** Sets the {@link Surface} and {@link Size} to render to. */
@VisibleForTesting
public void setVideoSurface(Surface surface, Size videoOutputSize) {
videoOutput = surface;
this.videoOutputSize = videoOutputSize;
@ -616,18 +605,12 @@ public final class CompositionPlayer extends SimpleBasePlayer
@Override
protected ListenableFuture<?> handleSetVideoOutput(Object videoOutput) {
if (!(videoOutput instanceof SurfaceHolder || videoOutput instanceof SurfaceView)) {
throw new UnsupportedOperationException(videoOutput.getClass().toString());
throw new UnsupportedOperationException(
videoOutput.getClass().toString()
+ ". Use CompositionPlayer.setVideoSurface() for Surface output.");
}
this.videoOutput = videoOutput;
if (composition == null) {
return Futures.immediateVoidFuture();
}
if (videoOutput instanceof SurfaceHolder) {
setVideoSurfaceHolderInternal((SurfaceHolder) videoOutput);
} else {
setVideoSurfaceHolderInternal(((SurfaceView) videoOutput).getHolder());
}
return Futures.immediateVoidFuture();
return maybeSetVideoOutput();
}
@Override
@ -1012,6 +995,25 @@ public final class CompositionPlayer extends SimpleBasePlayer
};
}
private ListenableFuture<?> maybeSetVideoOutput() {
if (videoOutput == null || composition == null) {
return Futures.immediateVoidFuture();
}
if (videoOutput instanceof SurfaceHolder) {
setVideoSurfaceHolderInternal((SurfaceHolder) videoOutput);
} else if (videoOutput instanceof SurfaceView) {
SurfaceView surfaceView = (SurfaceView) videoOutput;
setVideoSurfaceHolderInternal(surfaceView.getHolder());
} else if (videoOutput instanceof Surface) {
setVideoSurfaceInternal(
(Surface) videoOutput,
checkNotNull(videoOutputSize, "VideoOutputSize must be set when using Surface output"));
} else {
throw new IllegalStateException(videoOutput.getClass().toString());
}
return Futures.immediateVoidFuture();
}
private long getContentPositionMs() {
if (players.isEmpty()) {
return 0;