Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 880994b1 authored Oct 19, 2023 by Sungsoo Lim Committed by Automerger Merge Worker Oct 19, 2023

Merge "Preventing mHearingAidActiveDevices contains null" into main am:...

Merge "Preventing mHearingAidActiveDevices contains null" into main am: 5b834439 am: 0c3fa7dd am: a20abe9a

Original change: https://android-review.googlesource.com/c/platform/packages/modules/Bluetooth/+/2795912

Change-Id: I37eb5dee85b896255e41feb4ad928a08dd2d726d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

parents 15cf2221 a20abe9a

android/app/src/com/android/bluetooth/btservice/ActiveDeviceManager.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -676,7 +676,7 @@ public class ActiveDeviceManager implements AdapterService.BluetoothStateCallbac
		final HearingAidService hearingAidService = mFactory.getHearingAidService();
		if (hearingAidService != null) {
		long hiSyncId = hearingAidService.getHiSyncId(device);
		if (getHearingAidActiveHiSyncIdLocked() == hiSyncId) {
		if (device != null && getHearingAidActiveHiSyncIdLocked() == hiSyncId) {
		mHearingAidActiveDevices.add(device);
		} else {
		mHearingAidActiveDevices.clear();

android/app/src/com/android/bluetooth/btservice/AudioRoutingManager.java

+1 −1

Original line number	Diff line number	Diff line
		@@ -667,7 +667,7 @@ public class AudioRoutingManager extends ActiveDeviceManager {
		final HearingAidService hearingAidService = mFactory.getHearingAidService();
		if (hearingAidService != null) {
		long hiSyncId = hearingAidService.getHiSyncId(device);
		if (getHearingAidActiveHiSyncIdLocked() == hiSyncId) {
		if (device != null && getHearingAidActiveHiSyncIdLocked() == hiSyncId) {
		mHearingAidActiveDevices.add(device);
		} else {
		mHearingAidActiveDevices.clear();

android/app/tests/unit/src/com/android/bluetooth/btservice/ActiveDeviceManagerTest.java

+7 −0

Original line number	Diff line number	Diff line
		@@ -626,6 +626,13 @@ public class ActiveDeviceManagerTest {
		Assert.assertTrue(mActiveDeviceManager.getHearingAidActiveDevices().isEmpty());
		}

		@Test
		public void hearingAidActiveWithNull_clearHearingAidActiveDevices() {
		hearingAidActiveDeviceChanged(null);
		TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());
		assertThat(mActiveDeviceManager.getHearingAidActiveDevices()).isEmpty();
		}

		/**
		* One LE Audio is connected.
		*/

android/app/tests/unit/src/com/android/bluetooth/btservice/AudioRoutingManagerTest.java

+7 −0

Original line number	Diff line number	Diff line
		@@ -583,6 +583,13 @@ public class AudioRoutingManagerTest {
		assertThat(mAudioRoutingManager.getHearingAidActiveDevices().isEmpty()).isTrue();
		}

		@Test
		public void hearingAidActiveWithNull_clearHearingAidActiveDevices() {
		hearingAidActiveDeviceChanged(null);
		TestUtils.waitForLooperToFinishScheduledTask(mAudioRoutingManager.getHandlerLooper());
		assertThat(mAudioRoutingManager.getHearingAidActiveDevices()).isEmpty();
		}

		/** One LE Audio is connected. */
		@Test
		public void onlyLeAudioConnected_setHeadsetActive() {