From fe2975eff282cc394620bc33f2230d7fd526b86a Mon Sep 17 00:00:00 2001 From: JihCheng Chiu Date: Wed, 7 Nov 2018 22:21:52 +0800 Subject: DO NOT MERGE Change the logic of ContainerLayer's isVisible() The original logic of ContainerLayer's isVisible() checks the hidden flag. Therefore, SurfaceFlinger thought that those abstract surfaces have to show in display. However, they are meaningless for frame content. They should be ignored by SurfaceFlinger, so we can safely return false. Test: Change the type of abstract surface to ContainerLayer. Then slide UI, play video and so on. Bug: 111164627 Change-Id: I09e8647dc7fd8578d28dd79f3428a5e2eeb822fe --- services/surfaceflinger/ContainerLayer.cpp | 4 ---- 1 file changed, 4 deletions(-) (limited to 'services/surfaceflinger/ContainerLayer.cpp') diff --git a/services/surfaceflinger/ContainerLayer.cpp b/services/surfaceflinger/ContainerLayer.cpp index f259d93c3f..6f5bd0e3a4 100644 --- a/services/surfaceflinger/ContainerLayer.cpp +++ b/services/surfaceflinger/ContainerLayer.cpp @@ -30,10 +30,6 @@ ContainerLayer::ContainerLayer(SurfaceFlinger* flinger, const sp& client void ContainerLayer::onDraw(const RenderArea&, const Region& /* clip */, bool) const {} -bool ContainerLayer::isVisible() const { - return !isHiddenByPolicy(); -} - void ContainerLayer::setPerFrameData(const sp&) {} } // namespace android -- cgit v1.2.3-59-g8ed1b