tree 8c682f8acad769b1b9303d877100f4eda4f2a835
parent b1f8ccfb122bca5931aa0e1fdfb775f5d75fd225
author Soonil Nagarkar <sooniln@google.com> 1579648611 -0800
committer Soonil Nagarkar <sooniln@google.com> 1579713774 +0000

Overhaul FusedLocationProvider

Fix some minor bugs and ensure fused location provider correctly
supports location bypass. This is especially important for when
location bypass is invoked in direct boot.

The added UPDATE_DEVICE_STATS permission is necessary for FusedLocation
to correctly update WorkSources. FusedLocation receives work from LMS and
then further delegates that work to other location providers. The other
location providers should be informed of the correct applications for
battery blame, and should not be blaming the FusedLocation package.
1) This is the minimally scoped permission necessary to battery blame
correctly.
2) There is no way to attribute battery blame without this permission.
3) This is the correct permission - as required by LocationManager, and
this permission will likely never be removed (FusedLocation will always
need to battery blame).

Test: atest FusedLocationTests
Change-Id: If7126fffaae5577ddf8e366a0b5c17b3e5286582