diff options
| author | jwwang <wangchun@google.com> | 2019-08-27 15:39:19 +0800 |
|---|---|---|
| committer | jwwang <wangchun@google.com> | 2019-08-28 09:37:17 +0800 |
| commit | 1b85ccefebb121964c3e55ad3388abf801520e1f (patch) | |
| tree | b31830b75f6aed059a22f2904bfcdd55ce1610db /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | 007620d1b5bb3f473c13017409ced49271b98522 (diff) | |
Fix PackageWatchdog::syncRequestsAsync and related code.
A. mSomeHandler.post(this::doSomething);
B. mSomeHandler.removeCallbacks(this::doSomething);
B will not remove the pending runnable posted in A since each method reference will create a new Runnable instance.
Instead, we should do:
mDoSomething = this::doSomething;
mSomeHandler.post(mDoSomething);
mSomeHandler.removeCallbacks(mDoSomething);
Bug: 140085748
Test: Manual testing && atest PackageWatchdogTest
Change-Id: I15752e49741f4e3c3b894d925e2a58beab31209d
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions