From 6a91b2df51274709af1d319f7172a96ddf2bd860 Mon Sep 17 00:00:00 2001 From: huangdarwin Date: Mon, 20 Feb 2023 11:47:37 +0000 Subject: [PATCH] HDR: Remove comment explaining force sdr behavior with assetloaders. Before, if the upstream AssetLoader provides HDR to the VideoSamplePipeline when HDR_MODE_EXPERIMENTAL_FORCE_INTERPRET_HDR_AS_SDR is requested, the VideoSamplePipeline would attempt to tell the AssetLoader to output SDR, which could be accomplished via MediaCodec tone-mapping in the AssetLoader. However, this makes an assumption of the AssetLoader implementation, and AssetLoaders may not all implement support for decoder tone-mapping. Remove javadoc attempting to explain how AssetLoaders (ex. custom ones) could behave. PiperOrigin-RevId: 510956820 --- .../android/exoplayer2/transformer/Transformer.java | 8 -------- 1 file changed, 8 deletions(-) diff --git a/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java b/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java index 9009c60d5d..b1ed94e9a2 100644 --- a/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java +++ b/library/transformer/src/main/java/com/google/android/exoplayer2/transformer/Transformer.java @@ -136,14 +136,6 @@ public final class Transformer { * Listener#onFallbackApplied(Composition, TransformationRequest, TransformationRequest)} will * be invoked with the actual applied values. * - *

If {@link TransformationRequest#hdrMode} is set to {@link - * TransformationRequest#HDR_MODE_EXPERIMENTAL_FORCE_INTERPRET_HDR_AS_SDR} and an {@link - * #setAssetLoaderFactory AssetLoader.Factory} is set, {@link TransformationRequest#hdrMode} - * will be ignored. In this case, if the {@link AssetLoader.Factory} passed produces HDR data, - * it will be automatically {@linkplain - * TransformationRequest#HDR_MODE_TONE_MAP_HDR_TO_SDR_USING_MEDIACODEC tone mapped to SDR using - * MediaCodec}. - * * @param transformationRequest The {@link TransformationRequest}. * @return This builder. */