Throw if clip end position is equal to start position

Issue: #1242 PiperOrigin-RevId: 621850154
androidx · Apr 4, 2024 · 85d4df2 · 85d4df2
1 parent d57229a
commit 85d4df2
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/libraries/transformer/src/main/java/androidx/media3/transformer/EditedMediaItem.java b/libraries/transformer/src/main/java/androidx/media3/transformer/EditedMediaItem.java
@@ -25,6 +25,7 @@
 import androidx.media3.common.MediaItem;
 import androidx.media3.common.audio.AudioProcessor;
 import androidx.media3.common.util.UnstableApi;
+import androidx.media3.common.util.Util;
 import androidx.media3.exoplayer.source.MediaSource;
 import androidx.media3.extractor.mp4.Mp4Extractor;
 import com.google.errorprone.annotations.CanIgnoreReturnValue;
@@ -55,6 +56,14 @@ public static final class Builder {
      * @param mediaItem The {@link MediaItem} on which transformations are applied.
      */
     public Builder(MediaItem mediaItem) {
+      long clippingStartPositionUs = mediaItem.clippingConfiguration.startPositionUs;
+      long clippingEndPositionUs = mediaItem.clippingConfiguration.endPositionUs;
+      checkArgument(
+          clippingEndPositionUs == C.TIME_END_OF_SOURCE
+              || clippingEndPositionUs > clippingStartPositionUs,
+          Util.formatInvariant(
+              "Clipping end position (%d us) should be larger than start position (%d us)",
+              clippingEndPositionUs, clippingStartPositionUs));
       this.mediaItem = mediaItem;
       durationUs = C.TIME_UNSET;
       frameRate = C.RATE_UNSET_INT;