diff options
author | David Christie <dnchrist@google.com> | 2013-09-12 17:40:11 -0700 |
---|---|---|
committer | David Christie <dnchrist@google.com> | 2013-09-12 18:01:28 -0700 |
commit | 8b4cdbebcfb53b57ac2afec1b1e3377d6a02bb21 (patch) | |
tree | 02e5fed1041d4d2420b996a7b7a663528fedd7a1 /src/com/android/settings/location | |
parent | 54fa990f6174542e1e7fbe77a75954fcd2f39983 (diff) | |
download | packages_apps_Settings-8b4cdbebcfb53b57ac2afec1b1e3377d6a02bb21.zip packages_apps_Settings-8b4cdbebcfb53b57ac2afec1b1e3377d6a02bb21.tar.gz packages_apps_Settings-8b4cdbebcfb53b57ac2afec1b1e3377d6a02bb21.tar.bz2 |
Fix recent apps in appops.
Prior to this fix all apps would disappear from settings->location
15 minutes after the request was first made (even if request
was ongoing). Tested this out - apps requesting location longer
than the threshold stay in the list, and apps stay in the list exactly
the threshold time after the location request ENDS.
Change-Id: I7d4db25c615df2ec41e93b1b3191ba606d26719d
Diffstat (limited to 'src/com/android/settings/location')
-rw-r--r-- | src/com/android/settings/location/RecentLocationApps.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/com/android/settings/location/RecentLocationApps.java b/src/com/android/settings/location/RecentLocationApps.java index 1fa8aac..65fef80 100644 --- a/src/com/android/settings/location/RecentLocationApps.java +++ b/src/com/android/settings/location/RecentLocationApps.java @@ -194,9 +194,10 @@ public class RecentLocationApps { List<AppOpsManager.OpEntry> entries = ops.getOps(); boolean highBattery = false; boolean normalBattery = false; + // Earliest time for a location request to end and still be shown in list. + long recentLocationCutoffTime = now - RECENT_TIME_INTERVAL_MILLIS; for (AppOpsManager.OpEntry entry : entries) { - // If previous location activity is older than designated interval, ignore this app. - if (now - entry.getTime() <= RECENT_TIME_INTERVAL_MILLIS) { + if (entry.isRunning() || entry.getTime() >= recentLocationCutoffTime) { switch (entry.getOp()) { case AppOpsManager.OP_MONITOR_LOCATION: normalBattery = true; |