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

Merge "DO NOT MERGE Don't crash if already unregistered" into rvc-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2020-05-05 22:42:25 +00:00
committed by Android (Google) Code Review
parent c720603034 07c129cb1a
commit a9f4e2d2d7
1 changed files with 20 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
location/java/android/location/LocationManager.java
View File

@@ -2954,11 +2954,11 @@ public class LocationManager {
@Override @Override
protected void unregisterService() throws RemoteException { protected void unregisterService() throws RemoteException {
Preconditions.checkState(mListenerTransport != null); if (mListenerTransport != null) {
mService.unregisterGnssStatusCallback(mListenerTransport); mService.unregisterGnssStatusCallback(mListenerTransport);
mListenerTransport = null; mListenerTransport = null;
} }
}
private class GnssStatusListener extends IGnssStatusListener.Stub { private class GnssStatusListener extends IGnssStatusListener.Stub {
@Override @Override
@@ -3020,11 +3020,11 @@ public class LocationManager {
@Override @Override
protected void unregisterService() throws RemoteException { protected void unregisterService() throws RemoteException {
Preconditions.checkState(mListenerTransport != null); if (mListenerTransport != null) {
mService.removeGnssMeasurementsListener(mListenerTransport); mService.removeGnssMeasurementsListener(mListenerTransport);
mListenerTransport = null; mListenerTransport = null;
} }
}
@Override @Override
@Nullable @Nullable
@@ -3073,11 +3073,11 @@ public class LocationManager {
@Override @Override
protected void unregisterService() throws RemoteException { protected void unregisterService() throws RemoteException {
Preconditions.checkState(mListenerTransport != null); if (mListenerTransport != null) {
mService.removeGnssNavigationMessageListener(mListenerTransport); mService.removeGnssNavigationMessageListener(mListenerTransport);
mListenerTransport = null; mListenerTransport = null;
} }
}
private class GnssNavigationMessageListener extends IGnssNavigationMessageListener.Stub { private class GnssNavigationMessageListener extends IGnssNavigationMessageListener.Stub {
@Override @Override
@@ -3114,11 +3114,11 @@ public class LocationManager {
@Override @Override
protected void unregisterService() throws RemoteException { protected void unregisterService() throws RemoteException {
Preconditions.checkState(mListenerTransport != null); if (mListenerTransport != null) {
mService.removeGnssAntennaInfoListener(mListenerTransport); mService.removeGnssAntennaInfoListener(mListenerTransport);
mListenerTransport = null; mListenerTransport = null;
} }
}
private class GnssAntennaInfoListener extends IGnssAntennaInfoListener.Stub { private class GnssAntennaInfoListener extends IGnssAntennaInfoListener.Stub {
@Override @Override
@@ -3151,11 +3151,11 @@ public class LocationManager {
@Override @Override
protected void unregisterService() throws RemoteException { protected void unregisterService() throws RemoteException {
Preconditions.checkState(mListenerTransport != null); if (mListenerTransport != null) {
mService.removeGnssBatchingCallback(); mService.removeGnssBatchingCallback();
mListenerTransport = null; mListenerTransport = null;
} }
}
private class BatchedLocationCallback extends IBatchedLocationCallback.Stub { private class BatchedLocationCallback extends IBatchedLocationCallback.Stub {
@Override @Override