43a368dc19f8 - platform/frameworks/base

commit	43a368dc19f87669d3e319136d166b80d4f83d50	[log] [tgz]
author	Soonil Nagarkar <sooniln@google.com>	Mon Sep 21 13:36:24 2020 -0700
committer	Soonil Nagarkar <sooniln@google.com>	Mon Sep 21 14:15:47 2020 -0700
tree	5bea9740d65d002c3f6dbf0d61c3bdc4e5f2f744
parent	191ab1cb8a4a0372d35436ff4c8f6709cba7df9b [diff]

Fix bug in getCurrentLocation cancellation

Cancellation was not being propagated to the system server due to a
misunderstanding on how CancellationSignal functions. There is no
noticable impact for the user, as cancellation did properly prevent any
further locations, however this would have resulted in limited extra
power draw since the system server would not have canceled the
associated provider location request. Impact is limited since
getCurrentLocation() was limited to a 30s duration anyways.

Bug: 168666216
Test: manual + presubmits
Change-Id: I5c77db4cc56cce1b984587f65d2bcfb488436bb8

location/java/android/location/ILocationManager.aidl[diff]
location/java/android/location/LocationManager.java[diff]
services/core/java/com/android/server/location/LocationManagerService.java[diff]
services/core/java/com/android/server/location/LocationProviderManager.java[diff]
services/tests/mockingservicestests/src/com/android/server/location/LocationProviderManagerTest.java[diff]

5 files changed