commit 6753a021a7b134e7c68a0906871b84333bb9f4be
author tangrobin <tangrobin@google.com> Fri Aug 10 10:58:54 2018 +0800
committer Robin Tang <tangrobin@google.com> Fri Aug 10 07:39:40 2018 +0000
tree dabd23bd4c1a14e70dfa3453f2e3bd7117188d7a
parent 532bef184956ca5cde6c0717586153055d871de5

surfaceflinger: fix misplacement of getHwComposer related function

getHdrCapabilities and getSupportedPerFrameMetadata should not depend on hasWideColorDisplay.

Bug: 112082244
Test: screenrecord
Change-Id: I1c25d82eb41b5378b71f3837fe888012d43d9b1d

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 6b222d5..1b0064f 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -2328,7 +2328,9 @@
                     getHwComposer().getRenderIntents(displayId, colorMode);
             hwcColorModes.emplace(colorMode, renderIntents);
         }
+    }
 
+    if (displayId >= 0) {
         getHwComposer().getHdrCapabilities(displayId, &hdrCapabilities);
         supportedPerFrameMetadata = getHwComposer().getSupportedPerFrameMetadata(displayId);
     }