DO NOT MERGE Don't crash if already unregistered

Bug: 155284432
Test: manual
Change-Id: Id04ab9093b9cd4c837058442a8d0e04ff4dde051
This commit is contained in:
Soonil Nagarkar
2020-05-05 13:17:01 -07:00
parent ddb28a552f
commit 07c129cb1a

View File

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