summaryrefslogtreecommitdiff
path: root/services/surfaceflinger/Layer.cpp
diff options
context:
space:
mode:
author Robert Carr <racarr@google.com> 2018-12-10 13:01:14 -0800
committer Robert Carr <racarr@google.com> 2019-01-04 10:59:04 -0800
commitb89ea9d9533864bc6f73a24a4d33d3edba6d1365 (patch)
tree3286cbc9d7c2797ef168f5d85af21e83329823b0 /services/surfaceflinger/Layer.cpp
parent4db5fa02cf4c13901b1e3ae3f359efda6a62e91a (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.cpp12
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;