From 03b793e2ee1e901f8036289b0217dc3a26168476 Mon Sep 17 00:00:00 2001 From: sheenachhabra Date: Thu, 14 Sep 2023 09:52:13 -0700 Subject: [PATCH] Read muxedPartialVideo only in MUXER_MODE_MUX_PARTIAL_VIDEO mode When we switch from MUXER_MODE_MUX_PARTIAL_VIDEO to MUXER_MODE_APPEND_VIDEO `muxedPartialVideo` will already be `true` so `endTrack` method will pass through this `if(muxedPartialVideo)` check which is incorrect. PiperOrigin-RevId: 565398117 --- .../src/main/java/androidx/media3/transformer/MuxerWrapper.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libraries/transformer/src/main/java/androidx/media3/transformer/MuxerWrapper.java b/libraries/transformer/src/main/java/androidx/media3/transformer/MuxerWrapper.java index ba42f77f0e..ee59edbbd1 100644 --- a/libraries/transformer/src/main/java/androidx/media3/transformer/MuxerWrapper.java +++ b/libraries/transformer/src/main/java/androidx/media3/transformer/MuxerWrapper.java @@ -369,7 +369,7 @@ import org.checkerframework.checker.nullness.qual.RequiresNonNull; } } - if (muxedPartialVideo) { + if (muxerMode == MUXER_MODE_MUX_PARTIAL_VIDEO && muxedPartialVideo) { listener.onEnded(Util.usToMs(maxEndedTrackTimeUs), getCurrentOutputSizeBytes()); if (abortScheduledFuture != null) { abortScheduledFuture.cancel(/* mayInterruptIfRunning= */ false);