commit 13170c80f4ddb3f4cdb2ded601e0357fca68942e
author Peiyong Lin <lpy@google.com> Mon Jan 22 18:55:51 2018 -0800
committer Peiyong Lin <lpy@google.com> Tue Jan 23 15:10:01 2018 -0800
tree b0e80a34b529eb2af0211d74c0cedb21d2704f3a
parent 844e379081249acaf2b432a54a61686667e47e75

[SurfaceFlinger] Update ColorLayer to set dataspace.

This patch sets the dataspace for ColorLayer when setPerFrameData is called.
Previously, only the dataspace of BufferLayer is set. This helps clarify what
color space the color value that set through SetColor API is in.

BUG: 69970838
Test: None
Change-Id: I10982efc996bbecbcc433d3d7b0469ab8bf4cb5e

services/surfaceflinger/BufferLayer.cpp

diff --git a/services/surfaceflinger/BufferLayer.cpp b/services/surfaceflinger/BufferLayer.cpp
index d860f58..2ac4ec8 100644
--- a/services/surfaceflinger/BufferLayer.cpp
+++ b/services/surfaceflinger/BufferLayer.cpp
@@ -622,10 +622,10 @@
         setCompositionType(hwcId, HWC2::Composition::Device);
     }
 
-    ALOGV("setPerFrameData: dataspace = %d", mCurrentState.dataSpace);
-    error = hwcLayer->setDataspace(mCurrentState.dataSpace);
+    ALOGV("setPerFrameData: dataspace = %d", mDrawingState.dataSpace);
+    error = hwcLayer->setDataspace(mDrawingState.dataSpace);
     if (error != HWC2::Error::None) {
-        ALOGE("[%s] Failed to set dataspace %d: %s (%d)", mName.string(), mCurrentState.dataSpace,
+        ALOGE("[%s] Failed to set dataspace %d: %s (%d)", mName.string(), mDrawingState.dataSpace,
               to_string(error).c_str(), static_cast<int32_t>(error));
     }