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

Commit 1db78146 authored by Grace Jia's avatar Grace Jia Committed by Android (Google) Code Review
Browse files

Revert^2 "Fix ringtone play issue via LE devices." 

This reverts commit 11d0950a.

Reason for revert: fix b/31267505

Change-Id: Ica55561884ae5d5615a462f096e0193af7762271
parent 11d0950a

src/com/android/server/telecom/bluetooth/BluetoothDeviceManager.java

+3 −1
Original line number Diff line number Diff line
@@ -711,7 +711,9 @@ public class BluetoothDeviceManager { 
    }



    public boolean isInbandRingingEnabled() {

        BluetoothDevice activeDevice = mBluetoothRouteManager.getBluetoothAudioConnectedDevice();

        // Get the inband ringing enabled status of expected BT device to route call audio instead

        // of using the address of currently connected device.

        BluetoothDevice activeDevice = mBluetoothRouteManager.getMostRecentlyReportedActiveDevice();

        Log.i(this, "isInbandRingingEnabled: activeDevice: " + activeDevice);

        if (mBluetoothRouteManager.isCachedLeAudioDevice(activeDevice)) {

            if (mBluetoothLeAudioService == null) {

src/com/android/server/telecom/bluetooth/BluetoothRouteManager.java

+4 −0
Original line number Diff line number Diff line
@@ -652,6 +652,10 @@ public class BluetoothRouteManager extends StateMachine { 
        }

    }



    public BluetoothDevice getMostRecentlyReportedActiveDevice() {

        return mMostRecentlyReportedActiveDevice;

    }



    public boolean hasBtActiveDevice() {

        return mLeAudioActiveDeviceCache != null ||

                mHearingAidActiveDeviceCache != null ||

tests/src/com/android/server/telecom/tests/BluetoothDeviceManagerTest.java

+3 −8
Original line number Diff line number Diff line
@@ -755,20 +755,15 @@ public class BluetoothDeviceManagerTest extends TelecomTestCase { 
        when(mBluetoothHeadset.isInbandRingingEnabled()).thenReturn(false);

        when(mBluetoothLeAudio.isInbandRingtoneEnabled(1)).thenReturn(true);

        when(mBluetoothLeAudio.getGroupId(eq(device3))).thenReturn(1);

        receiverUnderTest.onReceive(mContext,

                buildConnectionActionIntent(BluetoothHeadset.STATE_CONNECTED, device1,

                        BluetoothDeviceManager.DEVICE_TYPE_HEADSET));

        receiverUnderTest.onReceive(mContext,

                buildConnectionActionIntent(BluetoothHeadset.STATE_CONNECTED, device2,

                        BluetoothDeviceManager.DEVICE_TYPE_HEADSET));

        receiverUnderTest.onReceive(mContext,

                buildConnectionActionIntent(BluetoothHeadset.STATE_CONNECTED, device3,

                        BluetoothDeviceManager.DEVICE_TYPE_LE_AUDIO));

        leAudioCallbacksTest.getValue().onGroupNodeAdded(device3, 1);

        when(mBluetoothLeAudio.getConnectedGroupLeadDevice(1)).thenReturn(device3);

        when(mRouteManager.getBluetoothAudioConnectedDevice()).thenReturn(device3);

        when(mRouteManager.isCachedLeAudioDevice(eq(device3))).thenReturn(true);

        assertEquals(3, mBluetoothDeviceManager.getNumConnectedDevices());

        when(mBluetoothLeAudio.getConnectedGroupLeadDevice(1)).thenReturn(device3);

        when(mRouteManager.getMostRecentlyReportedActiveDevice()).thenReturn(device3);

        assertEquals(1, mBluetoothDeviceManager.getNumConnectedDevices());

        assertTrue(mBluetoothDeviceManager.isInbandRingingEnabled());

    }