diff options
author | 2018-12-10 13:01:14 -0800 | |
---|---|---|
committer | 2019-01-04 10:59:04 -0800 | |
commit | b89ea9d9533864bc6f73a24a4d33d3edba6d1365 (patch) | |
tree | 3286cbc9d7c2797ef168f5d85af21e83329823b0 /services/surfaceflinger/Layer.cpp | |
parent | 4db5fa02cf4c13901b1e3ae3f359efda6a62e91a (diff) |
SurfaceFlinger: Removed createScopedConnection.
Scoped connections existed to constrain clients to only making
surfaces with parents. However now that we support off-screen parents
this is no longer required and we can use normal connections everywhere.
We take however care that only priviledged clients can place layers
in the current state.
Test: Manual
Bug: 62536731
Bug: 111373437
Bug: 111297488
Change-Id: I0a034767e92becec63071d7b1e3e71b95d505b77
Diffstat (limited to 'services/surfaceflinger/Layer.cpp')
-rw-r--r-- | services/surfaceflinger/Layer.cpp | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp index f4b3cddc63..2d3fd8ecbb 100644 --- a/services/surfaceflinger/Layer.cpp +++ b/services/surfaceflinger/Layer.cpp @@ -1666,11 +1666,6 @@ bool Layer::reparentChildren(const sp<IBinder>& newParentHandle) { } for (const sp<Layer>& child : mCurrentChildren) { newParent->addChild(child); - - sp<Client> client(child->mClientRef.promote()); - if (client != nullptr) { - client->updateParent(newParent); - } } mCurrentChildren.clear(); @@ -1705,13 +1700,6 @@ bool Layer::reparent(const sp<IBinder>& newParentHandle) { addToCurrentState(); } - sp<Client> client(mClientRef.promote()); - sp<Client> newParentClient(newParent->mClientRef.promote()); - - if (client != newParentClient) { - client->updateParent(newParent); - } - Mutex::Autolock lock(mStateMutex); if (mLayerDetached) { mLayerDetached = false; |