Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't hold wakelocks for passive requests

Browse Source 
Prevents power usage for passive requests and reduces wakelock
thrashing.

Bug: 206340085
Test: presubmits
Change-Id: I258ba16ace2a2a3fa248665bc936e6b855471762
This commit is contained in:
Soonil Nagarkar
2021-11-15 12:14:50 -08:00
parent cb4a25f6df
commit eca0b750e7
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
services/core/java/com/android/server/location/provider/LocationProviderManager.java
View File

@@ -911,18 +911,21 @@ public class LocationProviderManager extends
@Override @Override
public void onPreExecute() { public void onPreExecute() {
mUseWakeLock = false; mUseWakeLock = false;
final int size = locationResult.size();
for (int i = 0; i < size; ++i) { // don't acquire a wakelock for passive requests or for mock locations
if (!locationResult.get(i).isMock()) { if (getRequest().getIntervalMillis() != LocationRequest.PASSIVE_INTERVAL) {
mUseWakeLock = true; final int size = locationResult.size();
break; for (int i = 0; i < size; ++i) {
if (!locationResult.get(i).isMock()) {
mUseWakeLock = true;
break;
}
} }
} }
// update last delivered location // update last delivered location
setLastDeliveredLocation(locationResult.getLastLocation()); setLastDeliveredLocation(locationResult.getLastLocation());
// don't acquire a wakelock for mock locations to prevent abuse
if (mUseWakeLock) { if (mUseWakeLock) {
mWakeLock.acquire(WAKELOCK_TIMEOUT_MS); mWakeLock.acquire(WAKELOCK_TIMEOUT_MS);
} }