diff options
| author | wilsonshih <wilsonshih@google.com> | 2021-10-20 19:00:22 +0800 |
|---|---|---|
| committer | Wei Sheng Shih <wilsonshih@google.com> | 2021-11-12 03:14:07 +0000 |
| commit | d69deffd3551c7b008957763d218e6ed1b706384 (patch) | |
| tree | ef392b6a9a4e3d07280390dcdd9fd0fe55664dc1 /core/java/android/inputmethodservice/InputMethodService.java | |
| parent | c3af8564f98d2ccfc3a019fa83a3940c2c9b508d (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
Diffstat (limited to 'core/java/android/inputmethodservice/InputMethodService.java')
0 files changed, 0 insertions, 0 deletions