commit 23e07500a900bed481d87f0118392966244f36b8
author Arthur Hung <arthurhung@google.com> Fri Oct 15 11:58:19 2021 +0000
committer Arthur Hung <arthurhung@google.com> Thu Oct 21 14:31:39 2021 +0000
tree 796a6e9159e388edc316df35ffeb4211cbd87517
parent 4746c855fb82bda50ca5c4cccbcd84f264f0b4a4

Compute unique layer name in with sequence

Remove 'getUniqueLayerName' so we could remove stack lock from
'createLayer'.

Bug: 202621651
Test: atest libsurfaceflinger_unittest
Test: atest SurfaceFlinger_test
Change-Id: If5f4386b6da99bd540231b0c197c59a5823d7d4a

services/surfaceflinger/Layer.cpp

diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index 68ca2f0..3b98d50 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -88,7 +88,7 @@
 
 Layer::Layer(const LayerCreationArgs& args)
       : mFlinger(args.flinger),
-        mName(args.name),
+        mName(base::StringPrintf("%s#%d", args.name.c_str(), sequence)),
         mClientRef(args.client),
         mWindowType(
                 static_cast<WindowInfo::Type>(args.metadata.getInt32(METADATA_WINDOW_TYPE, 0))) {