From b9a3aa5ca1b7fb01fc6500a57d9ddcbba6ff9b54 Mon Sep 17 00:00:00 2001 From: huangdarwin Date: Fri, 30 Sep 2022 17:02:37 +0000 Subject: [PATCH] HDR: Add tests with HLG10 input. PiperOrigin-RevId: 478019046 --- .../transformer/mh/SetHdrEditingTest.java | 14 +++----------- .../transformer/mh/SetHdrToSdrToneMapTest.java | 1 - .../transformer/mh/TransformationTest.java | 5 +---- .../analysis/EncoderPerformanceAnalysisTest.java | 3 +-- 4 files changed, 5 insertions(+), 18 deletions(-) diff --git a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/SetHdrEditingTest.java b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/SetHdrEditingTest.java index 258e290cde..d73378a719 100644 --- a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/SetHdrEditingTest.java +++ b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/SetHdrEditingTest.java @@ -42,7 +42,6 @@ import java.util.concurrent.atomic.AtomicBoolean; import org.junit.Test; import org.junit.runner.RunWith; -// TODO(b/239172735): Add HLG tests after finding a shareable HLG file. /** {@link Transformer} instrumentation test for applying an HDR frame edit. */ @RunWith(AndroidJUnit4.class) public class SetHdrEditingTest { @@ -90,10 +89,7 @@ public class SetHdrEditingTest { String testId = "transformAndTranscode_hdr10File_whenHdrEditingIsSupported"; Context context = ApplicationProvider.getApplicationContext(); if (!deviceSupportsHdrEditing(VIDEO_H265, HDR10_DEFAULT_COLOR_INFO)) { - recordTestSkipped( - context, - testId, - /* reason= */ "Skipping on this device due to lack of HDR10 editing support."); + recordTestSkipped(context, testId, /* reason= */ "Device lacks HDR10 editing support."); return; } @@ -119,10 +115,7 @@ public class SetHdrEditingTest { String testId = "transformAndTranscode_hdr10File_toneMapsOrThrows_whenHdrEditingUnsupported"; Context context = ApplicationProvider.getApplicationContext(); if (deviceSupportsHdrEditing(VIDEO_H265, HDR10_DEFAULT_COLOR_INFO)) { - recordTestSkipped( - context, - testId, - /* reason= */ "Skipping on this device due to presence of HDR10 editing support."); + recordTestSkipped(context, testId, /* reason= */ "Device supports HDR10 editing."); return; } @@ -188,8 +181,7 @@ public class SetHdrEditingTest { recordTestSkipped( context, testId, - /* reason= */ "Skipping on this API version due to lack of support for" - + " MediaFormat#getInteger(String, int)."); + /* reason= */ "API version lacks support for MediaFormat#getInteger(String, int)."); return; } diff --git a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/SetHdrToSdrToneMapTest.java b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/SetHdrToSdrToneMapTest.java index 2f8c264b67..2ba107d7ce 100644 --- a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/SetHdrToSdrToneMapTest.java +++ b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/SetHdrToSdrToneMapTest.java @@ -36,7 +36,6 @@ import com.google.android.exoplayer2.util.Util; import org.junit.Test; import org.junit.runner.RunWith; -// TODO(b/239172735): Add HLG tests after finding a shareable HLG file. /** {@link Transformer} instrumentation test for applying an HDR to SDR tone mapping edit. */ @RunWith(AndroidJUnit4.class) public class SetHdrToSdrToneMapTest { diff --git a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/TransformationTest.java b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/TransformationTest.java index cedb49426f..43447aa276 100644 --- a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/TransformationTest.java +++ b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/TransformationTest.java @@ -174,10 +174,7 @@ public class TransformationTest { if (Util.SDK_INT < 25) { // TODO(b/210593256): Remove test skipping after removing the MediaMuxer dependency. - recordTestSkipped( - context, - testId, - /* reason= */ "Skipping on this API version due to lack of muxing support"); + recordTestSkipped(context, testId, /* reason= */ "API version lacks muxing support"); return; } diff --git a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/analysis/EncoderPerformanceAnalysisTest.java b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/analysis/EncoderPerformanceAnalysisTest.java index c63d497143..60d7e40c8b 100644 --- a/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/analysis/EncoderPerformanceAnalysisTest.java +++ b/library/transformer/src/androidTest/java/com/google/android/exoplayer2/transformer/mh/analysis/EncoderPerformanceAnalysisTest.java @@ -113,8 +113,7 @@ public class EncoderPerformanceAnalysisTest { recordTestSkipped( context, testId, - /* reason= */ "Skipping on this API version due to lack of support for setting operating" - + " rate and priority."); + /* reason= */ "API version lacks support for setting operating rate and priority."); return; }