Merge "Register the service-to-app callback whenever the service comes up for BluetoothConnectionCallback. This ensures that if the bluetooth process dies, the callbacks will be re-established once it comes back up."

This commit is contained in:
Treehugger Robot
2020-12-21 20:56:26 +00:00
committed by Gerrit Code Review

View File

@@ -2933,6 +2933,16 @@ public final class BluetoothAdapter {
}
});
}
synchronized (mBluetoothConnectionCallbackExecutorMap) {
if (!mBluetoothConnectionCallbackExecutorMap.isEmpty()) {
try {
mService.registerBluetoothConnectionCallback(mConnectionCallback);
} catch (RemoteException e) {
Log.e(TAG, "onBluetoothServiceUp: Failed to register bluetooth"
+ "connection callback", e);
}
}
}
}
public void onBluetoothServiceDown() {
@@ -3582,25 +3592,25 @@ public final class BluetoothAdapter {
return false;
}
// If the callback map is empty, we register the service-to-app callback
if (mBluetoothConnectionCallbackExecutorMap.isEmpty()) {
try {
mServiceLock.readLock().lock();
if (mService != null) {
if (!mService.registerBluetoothConnectionCallback(mConnectionCallback)) {
return false;
}
}
} catch (RemoteException e) {
Log.e(TAG, "", e);
mBluetoothConnectionCallbackExecutorMap.remove(callback);
} finally {
mServiceLock.readLock().unlock();
}
}
// Adds the passed in callback to our map of callbacks to executors
synchronized (mBluetoothConnectionCallbackExecutorMap) {
// If the callback map is empty, we register the service-to-app callback
if (mBluetoothConnectionCallbackExecutorMap.isEmpty()) {
try {
mServiceLock.readLock().lock();
if (mService != null) {
if (!mService.registerBluetoothConnectionCallback(mConnectionCallback)) {
return false;
}
}
} catch (RemoteException e) {
Log.e(TAG, "", e);
mBluetoothConnectionCallbackExecutorMap.remove(callback);
} finally {
mServiceLock.readLock().unlock();
}
}
// Adds the passed in callback to our map of callbacks to executors
if (mBluetoothConnectionCallbackExecutorMap.containsKey(callback)) {
throw new IllegalArgumentException("This callback has already been registered");
}