diff options
| author | Matthew Williams <mjwilliams@google.com> | 2014-09-23 13:39:36 -0700 |
|---|---|---|
| committer | Matthew Williams <mjwilliams@google.com> | 2014-09-23 17:34:46 -0700 |
| commit | 48a30db75dd0eedf8e065c89825b2af86a381b62 (patch) | |
| tree | 5fd56bfd2acd997a623f9603c64a29aa2c5ed113 /tools/aapt2/java/JavaClassGenerator_test.cpp | |
| parent | f7efd5c6a815b65e5401d51e0e9aae0f459eed72 (diff) | |
Fix lock ordering in JobScheduler
BUG: 17625667
Two part clean-up.
1) Don't try to lock in onControllerStateChanged. Do it in the handleMessage
instead where the rest of the locking is. This is sufficient to fix this bug.
2) The other side of the deadlock came b/c we lock when cancelling and calling
stopTrackingJob. Controllers handle their own locking so this isn't
necessary. B/c of a potential race from the controller side, added an explicit
check for the JSS to only run an expired job if it still exists.
Change-Id: Iaeebbc19437eb5b73e3ced3168f1fc13e564a4be
Diffstat (limited to 'tools/aapt2/java/JavaClassGenerator_test.cpp')
0 files changed, 0 insertions, 0 deletions