From bcbe3f1f80b58ea43fe5c7fc4b8d88ce2af63650 Mon Sep 17 00:00:00 2001 From: samrobinson Date: Wed, 15 Feb 2023 08:34:44 +0000 Subject: [PATCH] Clarify the mime type used in EncoderUtilTest. PiperOrigin-RevId: 509750806 --- .../transformer/EncoderUtilTest.java | 36 +++++++++---------- 1 file changed, 17 insertions(+), 19 deletions(-) diff --git a/library/transformer/src/test/java/com/google/android/exoplayer2/transformer/EncoderUtilTest.java b/library/transformer/src/test/java/com/google/android/exoplayer2/transformer/EncoderUtilTest.java index ef8fdb4e47..23271ba4e8 100644 --- a/library/transformer/src/test/java/com/google/android/exoplayer2/transformer/EncoderUtilTest.java +++ b/library/transformer/src/test/java/com/google/android/exoplayer2/transformer/EncoderUtilTest.java @@ -16,6 +16,7 @@ package com.google.android.exoplayer2.transformer; +import static com.google.android.exoplayer2.util.MimeTypes.VIDEO_H264; import static com.google.common.truth.Truth.assertThat; import android.media.MediaCodecInfo; @@ -24,7 +25,6 @@ import android.util.Size; import androidx.annotation.Nullable; import androidx.test.ext.junit.runners.AndroidJUnit4; import com.google.android.exoplayer2.C; -import com.google.android.exoplayer2.util.MimeTypes; import com.google.android.exoplayer2.video.ColorInfo; import com.google.common.collect.ImmutableList; import org.junit.After; @@ -45,12 +45,10 @@ import org.robolectric.shadows.ShadowMediaCodecList; */ @RunWith(AndroidJUnit4.class) public class EncoderUtilTest { - private static final String MIME_TYPE = MimeTypes.VIDEO_H264; - @Before public void setUp() { MediaFormat avcFormat = new MediaFormat(); - avcFormat.setString(MediaFormat.KEY_MIME, MediaFormat.MIMETYPE_VIDEO_AVC); + avcFormat.setString(MediaFormat.KEY_MIME, VIDEO_H264); MediaCodecInfo.CodecProfileLevel profileLevel = new MediaCodecInfo.CodecProfileLevel(); profileLevel.profile = MediaCodecInfo.CodecProfileLevel.AVCProfileHigh; // Using Level4 gives us 8192 16x16 blocks. If using width 1920 uses 120 blocks, 8192 / 120 = 68 @@ -81,12 +79,12 @@ public class EncoderUtilTest { @Test public void getSupportedResolution_withSupportedResolution_succeeds() { - ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(MIME_TYPE); + ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(VIDEO_H264); MediaCodecInfo encoderInfo = supportedEncoders.get(0); @Nullable Size closestSupportedResolution = - EncoderUtil.getSupportedResolution(encoderInfo, MIME_TYPE, 1920, 1080); + EncoderUtil.getSupportedResolution(encoderInfo, VIDEO_H264, 1920, 1080); assertThat(closestSupportedResolution).isNotNull(); assertThat(closestSupportedResolution.getWidth()).isEqualTo(1920); @@ -95,12 +93,12 @@ public class EncoderUtilTest { @Test public void getSupportedResolution_withUnalignedSize_findsMostCloselySupportedResolution() { - ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(MIME_TYPE); + ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(VIDEO_H264); MediaCodecInfo encoderInfo = supportedEncoders.get(0); @Nullable Size closestSupportedResolution = - EncoderUtil.getSupportedResolution(encoderInfo, MIME_TYPE, 1919, 1081); + EncoderUtil.getSupportedResolution(encoderInfo, VIDEO_H264, 1919, 1081); assertThat(closestSupportedResolution).isNotNull(); assertThat(closestSupportedResolution.getWidth()).isEqualTo(1920); @@ -110,12 +108,12 @@ public class EncoderUtilTest { @Test public void getSupportedResolution_findsThreeQuartersOfTheOriginalSize() { // The supported resolution will try to match the aspect ratio where possible. - ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(MIME_TYPE); + ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(VIDEO_H264); MediaCodecInfo encoderInfo = supportedEncoders.get(0); @Nullable Size closestSupportedResolution = - EncoderUtil.getSupportedResolution(encoderInfo, MIME_TYPE, 1920, 1920); + EncoderUtil.getSupportedResolution(encoderInfo, VIDEO_H264, 1920, 1920); assertThat(closestSupportedResolution).isNotNull(); assertThat(closestSupportedResolution.getWidth()).isEqualTo(1440); @@ -124,12 +122,12 @@ public class EncoderUtilTest { @Test public void getSupportedResolution_findsTwoThirdsOfTheOriginalSize() { - ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(MIME_TYPE); + ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(VIDEO_H264); MediaCodecInfo encoderInfo = supportedEncoders.get(0); @Nullable Size closestSupportedResolution = - EncoderUtil.getSupportedResolution(encoderInfo, MIME_TYPE, 2880, 1620); + EncoderUtil.getSupportedResolution(encoderInfo, VIDEO_H264, 2880, 1620); assertThat(closestSupportedResolution).isNotNull(); assertThat(closestSupportedResolution.getWidth()).isEqualTo(1920); @@ -138,12 +136,12 @@ public class EncoderUtilTest { @Test public void getSupportedResolution_findsHalfOfTheOriginalSize() { - ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(MIME_TYPE); + ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(VIDEO_H264); MediaCodecInfo encoderInfo = supportedEncoders.get(0); @Nullable Size closestSupportedResolution = - EncoderUtil.getSupportedResolution(encoderInfo, MIME_TYPE, 2160, 3840); + EncoderUtil.getSupportedResolution(encoderInfo, VIDEO_H264, 2160, 3840); assertThat(closestSupportedResolution).isNotNull(); assertThat(closestSupportedResolution.getWidth()).isEqualTo(1080); @@ -152,12 +150,12 @@ public class EncoderUtilTest { @Test public void getSupportedResolution_findsOneQuarterOfTheOriginalSize() { - ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(MIME_TYPE); + ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(VIDEO_H264); MediaCodecInfo encoderInfo = supportedEncoders.get(0); @Nullable Size closestSupportedResolution = - EncoderUtil.getSupportedResolution(encoderInfo, MIME_TYPE, 7680, 4320); + EncoderUtil.getSupportedResolution(encoderInfo, VIDEO_H264, 7680, 4320); assertThat(closestSupportedResolution).isNotNull(); assertThat(closestSupportedResolution.getWidth()).isEqualTo(1920); @@ -166,14 +164,14 @@ public class EncoderUtilTest { @Test public void getSupportedResolution_requestedReallyLarge_matchesAspectRatio() { - ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(MIME_TYPE); + ImmutableList supportedEncoders = EncoderUtil.getSupportedEncoders(VIDEO_H264); MediaCodecInfo encoderInfo = supportedEncoders.get(0); double aspectRatio = 1.5; @Nullable Size closestSupportedResolution = EncoderUtil.getSupportedResolution( - encoderInfo, MIME_TYPE, (int) (aspectRatio * 5000), 5000); + encoderInfo, VIDEO_H264, (int) (aspectRatio * 5000), 5000); assertThat(closestSupportedResolution).isNotNull(); assertThat( @@ -193,7 +191,7 @@ public class EncoderUtilTest { // support HDR. assertThat( EncoderUtil.getSupportedEncodersForHdrEditing( - MIME_TYPE, + VIDEO_H264, new ColorInfo.Builder() .setColorSpace(C.COLOR_SPACE_BT2020) .setColorRange(C.COLOR_RANGE_FULL)