Merge "Restart BT when init flags change" am: 9101fba8bd (83a1b877) · Commits · e / os / android_packages_modules_Bluetooth

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

+121 −74

Original line number	Diff line number	Diff line
		@@ -59,6 +59,7 @@ import android.os.UserHandle;
		import android.os.UserManager;
		import android.os.UserManagerInternal;
		import android.os.UserManagerInternal.UserRestrictionsListener;
		import android.provider.DeviceConfig;
		import android.provider.Settings;
		import android.provider.Settings.SettingNotFoundException;
		import android.text.TextUtils;
		@@ -109,6 +110,7 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		private static final int ADD_PROXY_DELAY_MS = 100;
		// Delay for retrying enable and disable in msec
		private static final int ENABLE_DISABLE_DELAY_MS = 300;
		private static final int DELAY_BEFORE_RESTART_DUE_TO_INIT_FLAGS_CHANGED_MS = 300;

		private static final int MESSAGE_ENABLE = 1;
		private static final int MESSAGE_DISABLE = 2;
		@@ -128,6 +130,7 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		private static final int MESSAGE_ADD_PROXY_DELAYED = 400;
		private static final int MESSAGE_BIND_PROFILE_SERVICE = 401;
		private static final int MESSAGE_RESTORE_USER_SETTING = 500;
		private static final int MESSAGE_INIT_FLAGS_CHANGED = 600;

		private static final int RESTORE_SETTING_TO_ON = 1;
		private static final int RESTORE_SETTING_TO_OFF = 0;
		@@ -267,6 +270,30 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		}
		};

		private final DeviceConfig.OnPropertiesChangedListener mDeviceConfigChangedListener =
		new DeviceConfig.OnPropertiesChangedListener() {
		@Override
		public void onPropertiesChanged(DeviceConfig.Properties properties) {
		if (!properties.getNamespace().equals(DeviceConfig.NAMESPACE_BLUETOOTH)) {
		return;
		}
		boolean foundInit = false;
		for (String name : properties.getKeyset()) {
		if (name.startsWith("INIT_")) {
		foundInit = true;
		break;
		}
		}
		if (!foundInit) {
		return;
		}
		mHandler.removeMessages(MESSAGE_INIT_FLAGS_CHANGED);
		mHandler.sendEmptyMessageDelayed(
		MESSAGE_INIT_FLAGS_CHANGED,
		DELAY_BEFORE_RESTART_DUE_TO_INIT_FLAGS_CHANGED_MS);
		}
		};

		public boolean onFactoryReset() {
		// Wait for stable state if bluetooth is temporary state.
		int state = getState();
		@@ -496,6 +523,10 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		Slog.w(TAG, "Unable to resolve SystemUI's UID.", e);
		}
		mSystemUiUid = systemUiUid;
		DeviceConfig.addOnPropertiesChangedListener(
		DeviceConfig.NAMESPACE_BLUETOOTH,
		(Runnable r) -> r.run(),
		mDeviceConfigChangedListener);
		}

		/**
		@@ -2108,6 +2139,50 @@ class BluetoothManagerService extends IBluetoothManager.Stub {

		/* disable and enable BT when detect a user switch */
		if (mBluetooth != null && isEnabled()) {
		restartForReason(BluetoothProtoEnums.ENABLE_DISABLE_REASON_USER_SWITCH);
		} else if (mBinding \|\| mBluetooth != null) {
		Message userMsg = mHandler.obtainMessage(MESSAGE_USER_SWITCHED);
		userMsg.arg2 = 1 + msg.arg2;
		// if user is switched when service is binding retry after a delay
		mHandler.sendMessageDelayed(userMsg, USER_SWITCHED_TIME_MS);
		if (DBG) {
		Slog.d(TAG, "Retry MESSAGE_USER_SWITCHED " + userMsg.arg2);
		}
		}
		break;
		}
		case MESSAGE_USER_UNLOCKED: {
		if (DBG) {
		Slog.d(TAG, "MESSAGE_USER_UNLOCKED");
		}
		mHandler.removeMessages(MESSAGE_USER_SWITCHED);

		if (mEnable && !mBinding && (mBluetooth == null)) {
		// We should be connected, but we gave up for some
		// reason; maybe the Bluetooth service wasn't encryption
		// aware, so try binding again.
		if (DBG) {
		Slog.d(TAG, "Enabled but not bound; retrying after unlock");
		}
		handleEnable(mQuietEnable);
		}
		break;
		}
		case MESSAGE_INIT_FLAGS_CHANGED: {
		if (DBG) {
		Slog.d(TAG, "MESSAGE_INIT_FLAGS_CHANGED");
		}
		mHandler.removeMessages(MESSAGE_INIT_FLAGS_CHANGED);
		if (mBluetooth != null && isEnabled()) {
		restartForReason(
		BluetoothProtoEnums.ENABLE_DISABLE_REASON_INIT_FLAGS_CHANGED);
		}
		break;
		}
		}
		}

		private void restartForReason(int reason) {
		try {
		mBluetoothLock.readLock().lock();
		if (mBluetooth != null) {
		@@ -2137,8 +2212,7 @@ class BluetoothManagerService extends IBluetoothManager.Stub {

		unbindAllBluetoothProfileServices();
		// disable
		addActiveLog(BluetoothProtoEnums.ENABLE_DISABLE_REASON_USER_SWITCH,
		mContext.getPackageName(), false);
		addActiveLog(reason, mContext.getPackageName(), false);
		handleDisable();
		// Pbap service need receive STATE_TURNING_OFF intent to close
		bluetoothStateChangeHandler(BluetoothAdapter.STATE_ON,
		@@ -2177,39 +2251,10 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		mHandler.removeMessages(MESSAGE_BLUETOOTH_STATE_CHANGE);
		mState = BluetoothAdapter.STATE_OFF;
		// enable
		addActiveLog(BluetoothProtoEnums.ENABLE_DISABLE_REASON_USER_SWITCH,
		mContext.getPackageName(), true);
		addActiveLog(reason, mContext.getPackageName(), true);
		// mEnable flag could have been reset on disableBLE. Reenable it.
		mEnable = true;
		handleEnable(mQuietEnable);
		} else if (mBinding \|\| mBluetooth != null) {
		Message userMsg = mHandler.obtainMessage(MESSAGE_USER_SWITCHED);
		userMsg.arg2 = 1 + msg.arg2;
		// if user is switched when service is binding retry after a delay
		mHandler.sendMessageDelayed(userMsg, USER_SWITCHED_TIME_MS);
		if (DBG) {
		Slog.d(TAG, "Retry MESSAGE_USER_SWITCHED " + userMsg.arg2);
		}
		}
		break;
		}
		case MESSAGE_USER_UNLOCKED: {
		if (DBG) {
		Slog.d(TAG, "MESSAGE_USER_UNLOCKED");
		}
		mHandler.removeMessages(MESSAGE_USER_SWITCHED);

		if (mEnable && !mBinding && (mBluetooth == null)) {
		// We should be connected, but we gave up for some
		// reason; maybe the Bluetooth service wasn't encryption
		// aware, so try binding again.
		if (DBG) {
		Slog.d(TAG, "Enabled but not bound; retrying after unlock");
		}
		handleEnable(mQuietEnable);
		}
		}
		}
		}
		}

		@@ -2643,6 +2688,8 @@ class BluetoothManagerService extends IBluetoothManager.Stub {
		return "RESTORE_USER_SETTING";
		case BluetoothProtoEnums.ENABLE_DISABLE_REASON_FACTORY_RESET:
		return "FACTORY_RESET";
		case BluetoothProtoEnums.ENABLE_DISABLE_REASON_INIT_FLAGS_CHANGED:
		return "INIT_FLAGS_CHANGED";
		case BluetoothProtoEnums.ENABLE_DISABLE_REASON_UNSPECIFIED:
		default: return "UNKNOWN[" + reason + "]";
		}