commit 4d9f6e666bb527b1c4dcd5f2e30be930d164322d
author Alec Mouri <alecmouri@google.com> Thu Dec 29 21:26:49 2022 +0000
committer Android (Google) Code Review <android-gerrit@google.com> Thu Dec 29 21:26:49 2022 +0000
tree 0a991f2204f87e4e382194740a555c5d0e4deb3b
parent eed9d5db739e2342c4219a2ef23f77097b3581f7

Revert "Treat extended range sRGB dataspaces as HDR"

This reverts commit eed9d5db739e2342c4219a2ef23f77097b3581f7.

Reason for revert: Possible TextureViewTest#testSDRFromSurfaceViewAndTextureView failures

Change-Id: I9b55c6339e54d71a22029c1325a95348aa7e52a4

libs/ui/tests/DataspaceUtils_test.cpp

diff --git a/libs/ui/tests/DataspaceUtils_test.cpp b/libs/ui/tests/DataspaceUtils_test.cpp
index ffe6438..3e09671 100644
--- a/libs/ui/tests/DataspaceUtils_test.cpp
+++ b/libs/ui/tests/DataspaceUtils_test.cpp
@@ -29,13 +29,12 @@
     EXPECT_TRUE(isHdrDataspace(ui::Dataspace::BT2020_ITU_PQ));
     EXPECT_TRUE(isHdrDataspace(ui::Dataspace::BT2020_PQ));
     EXPECT_TRUE(isHdrDataspace(ui::Dataspace::BT2020_HLG));
-    // The original formulation of scRGB indicates the same white points as that
-    // of sRGB, however scRGB may be used to implement HDR.
-    EXPECT_TRUE(isHdrDataspace(ui::Dataspace::V0_SCRGB_LINEAR));
-    EXPECT_TRUE(isHdrDataspace(ui::Dataspace::V0_SCRGB));
 
     EXPECT_FALSE(isHdrDataspace(ui::Dataspace::V0_SRGB_LINEAR));
+    // scRGB defines a very wide gamut but not an expanded luminance range
+    EXPECT_FALSE(isHdrDataspace(ui::Dataspace::V0_SCRGB_LINEAR));
     EXPECT_FALSE(isHdrDataspace(ui::Dataspace::V0_SRGB));
+    EXPECT_FALSE(isHdrDataspace(ui::Dataspace::V0_SCRGB));
     EXPECT_FALSE(isHdrDataspace(ui::Dataspace::V0_JFIF));
     EXPECT_FALSE(isHdrDataspace(ui::Dataspace::V0_BT601_625));
     EXPECT_FALSE(isHdrDataspace(ui::Dataspace::V0_BT601_525));