From fbeb59d10871df9fddea782b98c9520deda462f7 Mon Sep 17 00:00:00 2001 From: John Reck Date: Mon, 27 Mar 2023 18:01:09 -0400 Subject: [PATCH] Update DataSpace#RANGE_EXTENDED docs Bug: 274780844 Test: make offline-sdk-docs Change-Id: I0fa81462129321a03274aeffac49fd940f7bbf2d --- core/java/android/hardware/DataSpace.java | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/core/java/android/hardware/DataSpace.java b/core/java/android/hardware/DataSpace.java index b8b1eaaa164cd..312bfdf777d60 100644 --- a/core/java/android/hardware/DataSpace.java +++ b/core/java/android/hardware/DataSpace.java @@ -16,6 +16,7 @@ package android.hardware; import android.annotation.IntDef; +import android.view.SurfaceControl; import java.lang.annotation.Retention; import java.lang.annotation.RetentionPolicy; @@ -376,12 +377,19 @@ public final class DataSpace { */ public static final int RANGE_LIMITED = 2 << 27; /** - * Extended range is used for scRGB only. + * Extended range can be used in combination with FP16 to communicate scRGB or with + * {@link android.view.SurfaceControl.Transaction#setExtendedRangeBrightness(SurfaceControl, float, float)} + * to indicate an HDR range. * - *

Intended for use with floating point pixel formats. [0.0 - 1.0] is the standard - * sRGB space. Values outside the range [0.0 - 1.0] can encode - * color outside the sRGB gamut. [-0.5, 7.5] is the scRGB range. + *

When used with floating point pixel formats and #STANDARD_BT709 then [0.0 - 1.0] is the + * standard sRGB space and values outside the range [0.0 - 1.0] can encode + * color outside the sRGB gamut. [-0.5, 7.5] is the standard scRGB range. * Used to blend/merge multiple dataspaces on a single display.

+ * + *

As of {@link android.os.Build.VERSION_CODES#UPSIDE_DOWN_CAKE} this may be combined with + * {@link android.view.SurfaceControl.Transaction#setExtendedRangeBrightness(SurfaceControl, float, float)} + * and other formats such as {@link HardwareBuffer#RGBA_8888} or + * {@link HardwareBuffer#RGBA_1010102} to communicate a variable HDR brightness range

*/ public static final int RANGE_EXTENDED = 3 << 27;