diff options
| author | Sundeep Ghuman <sghuman@google.com> | 2017-02-08 17:19:27 -0800 |
|---|---|---|
| committer | Sundeep Ghuman <sghuman@google.com> | 2017-02-08 20:56:07 -0800 |
| commit | ac7b43616e8329fb19c1cfd6ec33637f4b0a1090 (patch) | |
| tree | 9e5bf64fee6cbabc014626b89efd404d117c2a98 /tools/aapt2/diff/Diff.cpp | |
| parent | 5a0d21156d4b74cc2a0772701a17a84e8a5051ae (diff) | |
Serially (un/)register score cache in WifiTracker.
Before the cache unregistration logic was posted to the worker thread.
However, when WifiSettings is quit, the onDestroy method immediately
kills the WorkHandler. This caused unregistration to never occur, which
left the cache registered and the listener to try to post work on the
dead worker thread. The cache is now registered and unregistered
serially in start and stop tracking, respectively. Furthermore, the
cachelistener is running on the mainHandler as it is a lightweight task
(posting to work handler).
Bug: 34891717
Test: runtest --path
frameworks/base/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/WifiTrackerTest.java
Change-Id: I7739fdbe036be5b2870be45f33f670fd69d69af1
Diffstat (limited to 'tools/aapt2/diff/Diff.cpp')
0 files changed, 0 insertions, 0 deletions