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

Commit 6546570d authored by Sungsoo Lim's avatar Sungsoo Lim
Browse files

Revert "Make BT audio routing more resemble the Android S behavior" 

This reverts commit 18d22e0e.

Reason for revert: b/262155254

Bug: 262155254
Bug: 247424095
Test: atest BluetoothInstrumentationTests
Change-Id: I32dca835c995bc4091d35bf16669b576f6af159e
parent 3f497536

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

+86 −316

File changed.

Preview size limit exceeded, changes collapsed.

android/app/src/com/android/bluetooth/mcp/MediaControlGattService.java

+10 −0
Original line number Diff line number Diff line
@@ -24,6 +24,8 @@ import static android.bluetooth.BluetoothGattCharacteristic.PROPERTY_READ; 
import static android.bluetooth.BluetoothGattCharacteristic.PROPERTY_WRITE;

import static android.bluetooth.BluetoothGattCharacteristic.PROPERTY_WRITE_NO_RESPONSE;



import static java.util.Map.entry;



import android.annotation.NonNull;

import android.annotation.Nullable;

import android.bluetooth.BluetoothDevice;
@@ -41,7 +43,9 @@ import android.os.Looper; 
import android.util.Log;

import android.util.Pair;



import com.android.bluetooth.a2dp.A2dpService;

import com.android.bluetooth.btservice.AdapterService;

import com.android.bluetooth.hearingaid.HearingAidService;

import com.android.bluetooth.le_audio.LeAudioService;

import com.android.internal.annotations.VisibleForTesting;
@@ -889,6 +893,12 @@ public class MediaControlGattService implements MediaControlGattServiceInterface 
        }



        if (req.getOpcode() == Request.Opcodes.PLAY) {

            if (mAdapterService.getActiveDevices(BluetoothProfile.A2DP).size() > 0) {

                A2dpService.getA2dpService().setActiveDevice(null);

            }

            if (mAdapterService.getActiveDevices(BluetoothProfile.HEARING_AID).size() > 0) {

                HearingAidService.getHearingAidService().setActiveDevice(null);

            }

            if (mLeAudioService == null) {

                mLeAudioService = LeAudioService.getLeAudioService();

            }

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

+12 −58
Original line number Diff line number Diff line
@@ -18,10 +18,10 @@ package com.android.bluetooth.btservice; 


import static org.mockito.Mockito.any;

import static org.mockito.Mockito.anyBoolean;

import static org.mockito.Mockito.doAnswer;

import static org.mockito.Mockito.isNull;

import static org.mockito.Mockito.never;

import static org.mockito.Mockito.timeout;

import static org.mockito.Mockito.times;

import static org.mockito.Mockito.verify;

import static org.mockito.Mockito.when;
@@ -60,6 +60,7 @@ import org.mockito.MockitoAnnotations; 


import java.util.ArrayList;

import java.util.List;

import java.util.Objects;



@MediumTest

@RunWith(AndroidJUnit4.class)
@@ -86,7 +87,6 @@ public class ActiveDeviceManagerTest { 
    @Mock private LeAudioService mLeAudioService;

    @Mock private AudioManager mAudioManager;

    @Mock private DatabaseManager mDatabaseManager;

    private AudioManager.OnModeChangedListener mAudioModeChangedListener;



    @Before

    public void setUp() throws Exception {
@@ -99,10 +99,6 @@ public class ActiveDeviceManagerTest { 
        MockitoAnnotations.initMocks(this);

        TestUtils.setAdapterService(mAdapterService);



        doAnswer(invocation -> {

            mAudioModeChangedListener = invocation.getArgument(1);

            return null;

        }).when(mAudioManager).addOnModeChangedListener(any(), any());

        when(mAdapterService.getSystemService(Context.AUDIO_SERVICE)).thenReturn(mAudioManager);

        when(mAdapterService.getSystemServiceName(AudioManager.class))

                .thenReturn(Context.AUDIO_SERVICE);
@@ -133,13 +129,15 @@ public class ActiveDeviceManagerTest { 
        when(mLeAudioService.setActiveDevice(any(), anyBoolean())).thenReturn(true);



        when(mA2dpService.getFallbackDevice()).thenAnswer(invocation -> {

            if (mDeviceConnectionStack.contains(mA2dpDevice)) {

            if (!mDeviceConnectionStack.isEmpty() && Objects.equals(mA2dpDevice,

                    mDeviceConnectionStack.get(mDeviceConnectionStack.size() - 1))) {

                return mA2dpDevice;

            }

            return null;

        });

        when(mHeadsetService.getFallbackDevice()).thenAnswer(invocation -> {

            if (mDeviceConnectionStack.contains(mHeadsetDevice)) {

            if (!mDeviceConnectionStack.isEmpty() && Objects.equals(mHeadsetDevice,

                    mDeviceConnectionStack.get(mDeviceConnectionStack.size() - 1))) {

                return mHeadsetDevice;

            }

            return null;
@@ -221,7 +219,7 @@ public class ActiveDeviceManagerTest { 
        a2dpActiveDeviceChanged(mA2dpDevice);

        // Don't call mA2dpService.setActiveDevice()

        TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());

        verify(mA2dpService).setActiveDevice(mA2dpDevice, false);

        verify(mA2dpService, times(1)).setActiveDevice(mA2dpDevice, false);

        Assert.assertEquals(mA2dpDevice, mActiveDeviceManager.getA2dpActiveDevice());

    }
@@ -297,7 +295,7 @@ public class ActiveDeviceManagerTest { 
        headsetActiveDeviceChanged(mHeadsetDevice);

        // Don't call mHeadsetService.setActiveDevice()

        TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());

        verify(mHeadsetService).setActiveDevice(mHeadsetDevice);

        verify(mHeadsetService, times(1)).setActiveDevice(mHeadsetDevice);

        Assert.assertEquals(mHeadsetDevice, mActiveDeviceManager.getHfpActiveDevice());

    }
@@ -368,7 +366,7 @@ public class ActiveDeviceManagerTest { 
        a2dpActiveDeviceChanged(mA2dpHeadsetDevice);



        TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());

        verify(mHearingAidService).setActiveDevice(null, true);

        verify(mHearingAidService).setActiveDevice(null, false);

        // Don't call mA2dpService.setActiveDevice()

        verify(mA2dpService, never()).setActiveDevice(mA2dpHeadsetDevice, false);

        Assert.assertEquals(mA2dpHeadsetDevice, mActiveDeviceManager.getA2dpActiveDevice());
@@ -388,7 +386,7 @@ public class ActiveDeviceManagerTest { 
        headsetActiveDeviceChanged(mA2dpHeadsetDevice);



        TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());

        verify(mHearingAidService).setActiveDevice(null, true);

        verify(mHearingAidService).setActiveDevice(null, false);

        // Don't call mHeadsetService.setActiveDevice()

        verify(mHeadsetService, never()).setActiveDevice(mA2dpHeadsetDevice);

        Assert.assertEquals(mA2dpHeadsetDevice, mActiveDeviceManager.getHfpActiveDevice());
@@ -442,7 +440,7 @@ public class ActiveDeviceManagerTest { 
        leAudioActiveDeviceChanged(mLeAudioDevice);

        // Don't call mLeAudioService.setActiveDevice()

        TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());

        verify(mLeAudioService).setActiveDevice(mLeAudioDevice, false);

        verify(mLeAudioService, times(1)).setActiveDevice(mLeAudioDevice, false);

        Assert.assertEquals(mLeAudioDevice, mActiveDeviceManager.getLeAudioActiveDevice());

    }
@@ -611,7 +609,7 @@ public class ActiveDeviceManagerTest { 
        a2dpActiveDeviceChanged(mA2dpDevice);

        TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());



        verify(mHearingAidService).setActiveDevice(null, true);

        verify(mHearingAidService).setActiveDevice(null, false);

        verify(mLeAudioService, never()).setActiveDevice(mLeAudioDevice);

        verify(mA2dpService, never()).setActiveDevice(mA2dpDevice, false);
@@ -695,50 +693,6 @@ public class ActiveDeviceManagerTest { 
        verify(mHearingAidService, timeout(TIMEOUT_MS)).setActiveDevice(null, false);

    }



    @Test

    public void leAudioActive_whileMediaPlayback_useLeAudioForMediaAndHeadsetForCall() {

        leAudioConnected(mLeAudioDevice);

        a2dpConnected(mA2dpDevice);

        headsetConnected(mHeadsetDevice);

        verify(mLeAudioService, timeout(TIMEOUT_MS)).setActiveDevice(mLeAudioDevice, false);

        verify(mA2dpService, timeout(TIMEOUT_MS)).setActiveDevice(mA2dpDevice, false);

        verify(mHeadsetService, timeout(TIMEOUT_MS)).setActiveDevice(mHeadsetDevice);



        Mockito.clearInvocations(mLeAudioService, mA2dpService, mHeadsetService);

        leAudioActiveDeviceChanged(mLeAudioDevice);

        TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());

        verify(mA2dpService).setActiveDevice(null, true);

        verify(mHeadsetService).setActiveDevice(null);



        Mockito.clearInvocations(mLeAudioService, mA2dpService, mHeadsetService);

        mAudioModeChangedListener.onModeChanged(AudioManager.MODE_IN_CALL);

        verify(mHeadsetService).setActiveDevice(mHeadsetDevice);

        verify(mLeAudioService).setActiveDevice(null, true);

    }



    @Test

    public void leAudioActive_whileInCall_useLeAudioForCallAndHeadsetForMedia() {

        when(mAudioManager.getMode()).thenReturn(AudioManager.MODE_IN_CALL);



        leAudioConnected(mLeAudioDevice);

        headsetConnected(mHeadsetDevice);

        a2dpConnected(mA2dpDevice);

        verify(mLeAudioService, timeout(TIMEOUT_MS)).setActiveDevice(mLeAudioDevice, false);

        verify(mA2dpService, timeout(TIMEOUT_MS)).setActiveDevice(mA2dpDevice, false);

        verify(mHeadsetService, timeout(TIMEOUT_MS)).setActiveDevice(mHeadsetDevice);



        Mockito.clearInvocations(mLeAudioService, mA2dpService, mHeadsetService);

        leAudioActiveDeviceChanged(mLeAudioDevice);

        TestUtils.waitForLooperToFinishScheduledTask(mActiveDeviceManager.getHandlerLooper());

        verify(mA2dpService).setActiveDevice(null, true);

        verify(mHeadsetService).setActiveDevice(null);



        Mockito.clearInvocations(mLeAudioService, mA2dpService, mHeadsetService);

        mAudioModeChangedListener.onModeChanged(AudioManager.MODE_NORMAL);

        verify(mA2dpService).setActiveDevice(mA2dpDevice, false);

        verify(mLeAudioService).setActiveDevice(null, true);

    }



    /**

     * Helper to indicate A2dp connected for a device.

     */