am 881e1a6b: Merge "do not request location from unavailable providers" into klp-dev

* commit '881e1a6b9101175efd027bcf3587c9f9c1dfbfaa':
  do not request location from unavailable providers
This commit is contained in:
Victoria Lease
2013-10-13 15:06:30 -07:00
committed by Android Git Automerger
2 changed files with 11 additions and 9 deletions

View File

@@ -131,6 +131,7 @@ public class FusionEngine implements LocationListener {
private void enableProvider(String name, long minTime) { private void enableProvider(String name, long minTime) {
ProviderStats stats = mStats.get(name); ProviderStats stats = mStats.get(name);
if (stats.available) {
if (!stats.requested) { if (!stats.requested) {
stats.requestTime = SystemClock.elapsedRealtime(); stats.requestTime = SystemClock.elapsedRealtime();
stats.requested = true; stats.requested = true;
@@ -141,6 +142,7 @@ public class FusionEngine implements LocationListener {
mLocationManager.requestLocationUpdates(name, minTime, 0, this, mLooper); mLocationManager.requestLocationUpdates(name, minTime, 0, this, mLooper);
} }
} }
}
private void disableProvider(String name) { private void disableProvider(String name) {
ProviderStats stats = mStats.get(name); ProviderStats stats = mStats.get(name);

View File

@@ -1473,7 +1473,7 @@ public class LocationManagerService extends ILocationManager.Stub {
+ " " + name + " " + request + " from " + packageName + "(" + uid + ")"); + " " + name + " " + request + " from " + packageName + "(" + uid + ")");
LocationProviderInterface provider = mProvidersByName.get(name); LocationProviderInterface provider = mProvidersByName.get(name);
if (provider == null) { if (provider == null) {
throw new IllegalArgumentException("provider doesn't exist: " + provider); throw new IllegalArgumentException("provider doesn't exist: " + name);
} }
UpdateRecord record = new UpdateRecord(name, request, receiver); UpdateRecord record = new UpdateRecord(name, request, receiver);