From efcb7683e55aabc69e9001e5f7e5aab1c1074240 Mon Sep 17 00:00:00 2001 From: claincly Date: Tue, 14 Feb 2023 10:28:33 +0000 Subject: [PATCH] Use ColorInfo.Builder in transformer and common. Because the ColorInfo constructor is deprecated. PiperOrigin-RevId: 509468663 --- .../exoplayer2/util/MediaFormatUtil.java | 7 ++++++- .../effect/GlEffectsFrameProcessor.java | 7 +++++-- .../transformer/AndroidTestUtil.java | 20 +++++++++---------- .../transformer/mh/HdrEditingTest.java | 20 +++++++++---------- .../transformer/VideoSamplePipeline.java | 10 +++++----- 5 files changed, 36 insertions(+), 28 deletions(-) diff --git a/library/common/src/main/java/com/google/android/exoplayer2/util/MediaFormatUtil.java b/library/common/src/main/java/com/google/android/exoplayer2/util/MediaFormatUtil.java index 5623fe5c62..420b2f305d 100644 --- a/library/common/src/main/java/com/google/android/exoplayer2/util/MediaFormatUtil.java +++ b/library/common/src/main/java/com/google/android/exoplayer2/util/MediaFormatUtil.java @@ -228,7 +228,12 @@ public final class MediaFormatUtil { || colorRange != Format.NO_VALUE || colorTransfer != Format.NO_VALUE || hdrStaticInfo != null) { - return new ColorInfo(colorSpace, colorRange, colorTransfer, hdrStaticInfo); + return new ColorInfo.Builder() + .setColorSpace(colorSpace) + .setColorRange(colorRange) + .setColorTransfer(colorTransfer) + .setHdrStaticInfo(hdrStaticInfo) + .build(); } return null; } diff --git a/library/effect/src/main/java/com/google/android/exoplayer2/effect/GlEffectsFrameProcessor.java b/library/effect/src/main/java/com/google/android/exoplayer2/effect/GlEffectsFrameProcessor.java index ab0b217ddd..2248f2888c 100644 --- a/library/effect/src/main/java/com/google/android/exoplayer2/effect/GlEffectsFrameProcessor.java +++ b/library/effect/src/main/java/com/google/android/exoplayer2/effect/GlEffectsFrameProcessor.java @@ -258,8 +258,11 @@ public final class GlEffectsFrameProcessor implements FrameProcessor { ImmutableList.Builder rgbMatrixListBuilder = new ImmutableList.Builder<>(); boolean sampleFromInputTexture = true; ColorInfo linearColorInfo = - new ColorInfo( - outputColorInfo.colorSpace, outputColorInfo.colorRange, C.COLOR_TRANSFER_LINEAR, null); + outputColorInfo + .buildUpon() + .setColorTransfer(C.COLOR_TRANSFER_LINEAR) + .setHdrStaticInfo(null) + .build(); for (int i = 0; i < effects.size(); i++) { Effect effect = effects.get(i); checkArgument(effect instanceof GlEffect, "GlEffectsFrameProcessor only supports GlEffects"); diff --git a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/AndroidTestUtil.java b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/AndroidTestUtil.java index 87c7903026..c0afd0dc0e 100644 --- a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/AndroidTestUtil.java +++ b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/AndroidTestUtil.java @@ -103,11 +103,11 @@ public final class AndroidTestUtil { .setHeight(1080) .setFrameRate(30.000f) .setColorInfo( - new ColorInfo( - C.COLOR_SPACE_BT2020, - C.COLOR_RANGE_LIMITED, - C.COLOR_TRANSFER_HLG, - /* hdrStaticInfo= */ null)) + new ColorInfo.Builder() + .setColorSpace(C.COLOR_SPACE_BT2020) + .setColorRange(C.COLOR_RANGE_LIMITED) + .setColorTransfer(C.COLOR_TRANSFER_HLG) + .build()) .build(); public static final String MP4_ASSET_1080P_4_SECOND_HDR10 = "asset:///media/mp4/hdr10-1080p.mp4"; public static final Format MP4_ASSET_1080P_4_SECOND_HDR10_FORMAT = @@ -117,11 +117,11 @@ public final class AndroidTestUtil { .setHeight(1080) .setFrameRate(23.517f) .setColorInfo( - new ColorInfo( - C.COLOR_SPACE_BT2020, - C.COLOR_RANGE_LIMITED, - C.COLOR_TRANSFER_ST2084, - /* hdrStaticInfo= */ null)) + new ColorInfo.Builder() + .setColorSpace(C.COLOR_SPACE_BT2020) + .setColorRange(C.COLOR_RANGE_LIMITED) + .setColorTransfer(C.COLOR_TRANSFER_ST2084) + .build()) .build(); public static final String MP4_ASSET_1080P_1_SECOND_HDR10_VIDEO_SDR_CONTAINER = "asset:///media/mp4/hdr10-video-with-sdr-container.mp4"; diff --git a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/HdrEditingTest.java b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/HdrEditingTest.java index d5f02999cb..b57072ba01 100644 --- a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/HdrEditingTest.java +++ b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/HdrEditingTest.java @@ -56,17 +56,17 @@ import org.junit.runner.RunWith; public class HdrEditingTest { public static final String TAG = "HdrEditingTest"; private static final ColorInfo HDR10_DEFAULT_COLOR_INFO = - new ColorInfo( - C.COLOR_SPACE_BT2020, - C.COLOR_RANGE_LIMITED, - C.COLOR_TRANSFER_ST2084, - /* hdrStaticInfo= */ null); + new ColorInfo.Builder() + .setColorSpace(C.COLOR_SPACE_BT2020) + .setColorRange(C.COLOR_RANGE_LIMITED) + .setColorTransfer(C.COLOR_TRANSFER_ST2084) + .build(); private static final ColorInfo HLG10_DEFAULT_COLOR_INFO = - new ColorInfo( - C.COLOR_SPACE_BT2020, - C.COLOR_RANGE_LIMITED, - C.COLOR_TRANSFER_HLG, - /* hdrStaticInfo= */ null); + new ColorInfo.Builder() + .setColorSpace(C.COLOR_SPACE_BT2020) + .setColorRange(C.COLOR_RANGE_LIMITED) + .setColorTransfer(C.COLOR_TRANSFER_HLG) + .build(); @Test public void transform_noRequestedTranscode_hdr10File_transformsOrThrows() throws Exception { diff --git a/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/VideoSamplePipeline.java b/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/VideoSamplePipeline.java index 11870ac5b2..b28a788db2 100644 --- a/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/VideoSamplePipeline.java +++ b/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/VideoSamplePipeline.java @@ -128,11 +128,11 @@ import org.checkerframework.dataflow.qual.Pure; // C.COLOR_TRANSFER_SDR to the encoder. ColorInfo frameProcessorOutputColor = isGlToneMapping - ? new ColorInfo( - C.COLOR_SPACE_BT709, - C.COLOR_RANGE_LIMITED, - C.COLOR_TRANSFER_GAMMA_2_2, - /* hdrStaticInfo= */ null) + ? new ColorInfo.Builder() + .setColorSpace(C.COLOR_SPACE_BT709) + .setColorRange(C.COLOR_RANGE_LIMITED) + .setColorTransfer(C.COLOR_TRANSFER_GAMMA_2_2) + .build() : encoderInputColor; try { frameProcessor =