Merge "Synchronize adapter registration with mCallbacks" am: a37b682ee4 am: 47359f8f69 (7f9a7058) · Commits · e / os / android_packages_modules_Bluetooth

service/java/com/android/server/bluetooth/BluetoothManagerService.java

+30 −40

Original line number	Diff line number	Diff line
		@@ -119,8 +119,6 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		private static final int MESSAGE_DISABLE = 2;
		private static final int MESSAGE_HANDLE_ENABLE_DELAYED = 3;
		private static final int MESSAGE_HANDLE_DISABLE_DELAYED = 4;
		private static final int MESSAGE_REGISTER_ADAPTER = 20;
		private static final int MESSAGE_UNREGISTER_ADAPTER = 21;
		private static final int MESSAGE_REGISTER_STATE_CHANGE_CALLBACK = 30;
		private static final int MESSAGE_UNREGISTER_STATE_CHANGE_CALLBACK = 31;
		private static final int MESSAGE_BLUETOOTH_SERVICE_CONNECTED = 40;
		@@ -642,10 +640,9 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		Slog.w(TAG, "Callback is null in registerAdapter");
		return null;
		}
		Message msg = mHandler.obtainMessage(MESSAGE_REGISTER_ADAPTER);
		msg.obj = callback;
		mHandler.sendMessage(msg);

		synchronized (mCallbacks) {
		mCallbacks.register(callback);
		}
		return mBluetooth;
		}

		@@ -655,9 +652,9 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		return;
		}
		mContext.enforceCallingOrSelfPermission(BLUETOOTH_PERM, "Need BLUETOOTH permission");
		Message msg = mHandler.obtainMessage(MESSAGE_UNREGISTER_ADAPTER);
		msg.obj = callback;
		mHandler.sendMessage(msg);
		synchronized (mCallbacks) {
		mCallbacks.unregister(callback);
		}
		}

		public void registerStateChangeCallback(IBluetoothStateChangeCallback callback) {
		@@ -1559,6 +1556,7 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		* Inform BluetoothAdapter instances that Adapter service is up
		*/
		private void sendBluetoothServiceUpCallback() {
		synchronized (mCallbacks) {
		try {
		int n = mCallbacks.beginBroadcast();
		Slog.d(TAG, "Broadcasting onBluetoothServiceUp() to " + n + " receivers.");
		@@ -1573,11 +1571,13 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		mCallbacks.finishBroadcast();
		}
		}
		}

		/**
		* Inform BluetoothAdapter instances that Adapter service is down
		*/
		private void sendBluetoothServiceDownCallback() {
		synchronized (mCallbacks) {
		try {
		int n = mCallbacks.beginBroadcast();
		Slog.d(TAG, "Broadcasting onBluetoothServiceDown() to " + n + " receivers.");
		@@ -1592,6 +1592,7 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		mCallbacks.finishBroadcast();
		}
		}
		}

		public String getAddress() {
		mContext.enforceCallingOrSelfPermission(BLUETOOTH_PERM, "Need BLUETOOTH permission");
		@@ -1917,17 +1918,6 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		mContext.getPackageName());
		}
		break;

		case MESSAGE_REGISTER_ADAPTER: {
		IBluetoothManagerCallback callback = (IBluetoothManagerCallback) msg.obj;
		mCallbacks.register(callback);
		break;
		}
		case MESSAGE_UNREGISTER_ADAPTER: {
		IBluetoothManagerCallback callback = (IBluetoothManagerCallback) msg.obj;
		mCallbacks.unregister(callback);
		break;
		}
		case MESSAGE_REGISTER_STATE_CHANGE_CALLBACK: {
		IBluetoothStateChangeCallback callback =
		(IBluetoothStateChangeCallback) msg.obj;