diff options
| author | chaviw <chaviw@google.com> | 2020-02-19 16:00:08 -0800 |
|---|---|---|
| committer | Chavi Weingarten <chaviw@google.com> | 2020-02-24 18:09:51 +0000 |
| commit | b9c44aae79938ff7f979254945cc1f66c09bf5cd (patch) | |
| tree | c108ad34a56b36c6102b2642de063085799c9ee5 /tools/aapt2/java/JavaClassGenerator.cpp | |
| parent | fd2f367df6f6e351ef5660511946d7bd7e6c3e8a (diff) | |
Fix Display SurfaceControl leak
The DisplayContent was merging its transaction into the
RootWindowContainer's pendingTransaction. However, that transaction is
never applied so it was stuck in that transaction forever.
There's no need to merge to parent transaction when removing since
calling getPendingTransaction will already return the DisplayContent's
transaction.
Also added a forced Transaction.apply() when removing the DisplayContent
since there's a chance it may no be reachable when
performSurfacePlacement is called to apply all transactions.
Test: EncodeVirtualDisplayWithCompositionTest#testVirtualDisplayRecycles
No longer leaks SurfaceControls
Fixes: 149526716
Change-Id: I14aa5b2e0e14d75fb0285a1b7a511235064f621c
(cherry picked from commit 93938469fc7866cc199f136657b575d1d7c5dae1)
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator.cpp')
0 files changed, 0 insertions, 0 deletions