e731ca8931cba465b03ea109ddc304a127a909d3 - platform_frameworks_base

commit	e731ca8931cba465b03ea109ddc304a127a909d3	[log] [tgz]
author	Soonil Nagarkar <sooniln@google.com>	Fri Nov 02 13:55:51 2018 -0700
committer	Soonil Nagarkar <sooniln@google.com>	Wed Nov 07 12:22:20 2018 -0800
tree	37743e2f036b0f7d6314029c5d2e8fab70688a5e
parent	20e5558d66e4fa63a6c976f69a69e9be0484e8b5 [diff]

Refactor ServiceWatcher

Make thread safe and lock free and fixes a variety of minor bugs. Most
of location code is under the synchronization strategy of 'throw
everything behind one massive lock'. This is a bad idea for a variety
of reasons and makes it very difficult to reason about thread safety.
Correcting this will make code easier to read and understand.

Test: Tested manually on device.

Change-Id: I42ad9b4009eabc04227008d6d562dca557087d1f

services/core/java/com/android/server/LocationManagerService.java[diff]
services/core/java/com/android/server/ServiceWatcher.java[diff]
services/core/java/com/android/server/location/ActivityRecognitionProxy.java[diff]
services/core/java/com/android/server/location/GeocoderProxy.java[diff]
services/core/java/com/android/server/location/GeofenceProxy.java[diff]
services/core/java/com/android/server/location/LocationProviderProxy.java[diff]

6 files changed

tree: 37743e2f036b0f7d6314029c5d2e8fab70688a5e