diff options
| author | wilsonshih <wilsonshih@google.com> | 2021-10-20 19:00:22 +0800 |
|---|---|---|
| committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2021-11-24 23:22:19 +0000 |
| commit | 05e7906edb106fc72b0cba5b1e3a936503ee7fc8 (patch) | |
| tree | dc77ab4a711f8e2b4fd5906dfb6e8a30ddc0d454 /tools/aapt2/diff/Diff.cpp | |
| parent | 8347f70f27f5efbaa6eeba72f83c8a4f47dac413 (diff) | |
Only register listeners if setView was success.
Found a memory leak on ViewRootImpl. If the view wasn't been set
to the VRI successfully, there would keep the registed listeners to the
AccessibilityManager and DisplayManager forever because there didn't
need to deatch anything from window.
Another reasonable way is to register all listeners after setView
success.
Bug: 200843755
Test: manual, hard code to force addWindow fail and monitor no more
ViewRootImpl object leaked to AccessibilityManager and DisplayManager.
Change-Id: I3c9cc52d0ca4595c74c1dc3c51d286d9e6e3897f
Merged-In: I3c9cc52d0ca4595c74c1dc3c51d286d9e6e3897f
(cherry picked from commit d69deffd3551c7b008957763d218e6ed1b706384)
Merged-In:I3c9cc52d0ca4595c74c1dc3c51d286d9e6e3897f
Diffstat (limited to 'tools/aapt2/diff/Diff.cpp')
0 files changed, 0 insertions, 0 deletions