diff options
| author | Hugo Benichi <hugobenichi@google.com> | 2016-07-07 09:36:12 +0900 |
|---|---|---|
| committer | Hugo Benichi <hugobenichi@google.com> | 2016-07-07 09:44:02 +0900 |
| commit | 39e10e88ffe351090346508045876d59c1370cd9 (patch) | |
| tree | e12ab16ef990c590dbfb70ba6ceb4b63e6d4448d /tools/aapt2/diff/Diff.cpp | |
| parent | 018a2058246d1f034d25adbd69502e6b2eb81fc7 (diff) | |
ConnectivityManager: no double callback remove
This patch partially undoes ag/869831 (Change-Id:
Ia42ed7aefaebd8caf3eada8e42b6cb7a940d7647) so that ConnectivityManager
does not remove callbacks from its internal request-to-callback map at
unregistration, but instead let the singleton CallbackHandler do it when
receiving a CALLBACK_RELEASED from ConnectivityService.
ag/869831 was thought to fix b/26749700 that reported a callback leak
from sNetworkCallback, but a finer analysis of the code shows that
callbacks were correctly removed by the CallbackHandler before
ag/869831. There was therefore no callback leak.
Bug: 26749700
Bug: 28537383
Change-Id: I421d889d0e225c0e3d1eebea664f44a1cc0f3191
Diffstat (limited to 'tools/aapt2/diff/Diff.cpp')
0 files changed, 0 insertions, 0 deletions