Merge \\"Fix data race\\" into nyc-dev am: 838dfe60f0
am: 6aa4b7fb11
Change-Id: Ieaa3e4a4ba06f9786357e0b78f097315e12910b6
This commit is contained in:
@@ -40,7 +40,6 @@ public class BluetoothControllerImpl implements BluetoothController, BluetoothCa
|
||||
private static final String TAG = "BluetoothController";
|
||||
private static final boolean DEBUG = Log.isLoggable(TAG, Log.DEBUG);
|
||||
|
||||
private final ArrayList<Callback> mCallbacks = new ArrayList<Callback>();
|
||||
private final LocalBluetoothManager mLocalBluetoothManager;
|
||||
private final UserManager mUserManager;
|
||||
private final int mCurrentUser;
|
||||
@@ -78,7 +77,7 @@ public class BluetoothControllerImpl implements BluetoothController, BluetoothCa
|
||||
pw.print(" mEnabled="); pw.println(mEnabled);
|
||||
pw.print(" mConnectionState="); pw.println(stateToString(mConnectionState));
|
||||
pw.print(" mLastDevice="); pw.println(mLastDevice);
|
||||
pw.print(" mCallbacks.size="); pw.println(mCallbacks.size());
|
||||
pw.print(" mCallbacks.size="); pw.println(mHandler.mCallbacks.size());
|
||||
pw.println(" Bluetooth Devices:");
|
||||
for (CachedBluetoothDevice device :
|
||||
mLocalBluetoothManager.getCachedDeviceManager().getCachedDevicesCopy()) {
|
||||
@@ -106,13 +105,13 @@ public class BluetoothControllerImpl implements BluetoothController, BluetoothCa
|
||||
|
||||
@Override
|
||||
public void addStateChangedCallback(Callback cb) {
|
||||
mCallbacks.add(cb);
|
||||
mHandler.obtainMessage(H.MSG_ADD_CALLBACK, cb).sendToTarget();
|
||||
mHandler.sendEmptyMessage(H.MSG_STATE_CHANGED);
|
||||
}
|
||||
|
||||
@Override
|
||||
public void removeStateChangedCallback(Callback cb) {
|
||||
mCallbacks.remove(cb);
|
||||
mHandler.obtainMessage(H.MSG_REMOVE_CALLBACK, cb).sendToTarget();
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -236,8 +235,12 @@ public class BluetoothControllerImpl implements BluetoothController, BluetoothCa
|
||||
}
|
||||
|
||||
private final class H extends Handler {
|
||||
private final ArrayList<BluetoothController.Callback> mCallbacks = new ArrayList<>();
|
||||
|
||||
private static final int MSG_PAIRED_DEVICES_CHANGED = 1;
|
||||
private static final int MSG_STATE_CHANGED = 2;
|
||||
private static final int MSG_ADD_CALLBACK = 3;
|
||||
private static final int MSG_REMOVE_CALLBACK = 4;
|
||||
|
||||
@Override
|
||||
public void handleMessage(Message msg) {
|
||||
@@ -248,6 +251,12 @@ public class BluetoothControllerImpl implements BluetoothController, BluetoothCa
|
||||
case MSG_STATE_CHANGED:
|
||||
fireStateChange();
|
||||
break;
|
||||
case MSG_ADD_CALLBACK:
|
||||
mCallbacks.add((BluetoothController.Callback) msg.obj);
|
||||
break;
|
||||
case MSG_REMOVE_CALLBACK:
|
||||
mCallbacks.remove((BluetoothController.Callback) msg.obj);
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user