Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e9722be4 authored by Melody Hsu's avatar Melody Hsu
Browse files

Fix set initialization for snapshot requests 

The set for exclude layer IDs was always initialized with 1 element
containing the value "0" because of the double curly braces used in the
snapshot request initialization. This led to region sampling always
doing additional unnecessary work when getting layer snapshots since the
content of the set is used to determine which "getSnapshot" path is
taken.

Bug: b/417069600
Test: logging, traces
Flag: EXEMPT, bug fix
Change-Id: I79e1b012888c4210e525896e465f3b9780a9937c
parent 187edc86

services/surfaceflinger/SurfaceFlinger.h

+1 −1
Original line number Diff line number Diff line
@@ -890,7 +890,7 @@ private: 
        std::optional<uint32_t> rootLayerId{std::nullopt};



        // IDs of layers that will be excluded from the screenshot

        std::unordered_set<uint32_t> excludeLayerIds{{}};

        std::unordered_set<uint32_t> excludeLayerIds{};



        // If true, transform is inverted from the parent layer snapshot

        bool childrenOnly{false};