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

Commit 134f5593 authored by Cam Bickel's avatar Cam Bickel Committed by Camden Bickel
Browse files

audio: On desktop, show output device USB name 

Bug: b/366358858
Flag: com.android.media.flags.enable_audio_input_device_routing_and_volume_control
Test: atest PhoneMediaDeviceTest
Change-Id: I7a3d08e28deb997e2bec36af41a84d45fec5f4f8
parent 67d0d36b

packages/SettingsLib/src/com/android/settingslib/media/PhoneMediaDevice.java

+2 −2
Original line number Diff line number Diff line
@@ -41,9 +41,9 @@ import android.media.RouteListingPreference; 
import android.os.SystemProperties;

import android.util.Log;



import androidx.annotation.VisibleForTesting;

import androidx.annotation.NonNull;

import androidx.annotation.Nullable;

import androidx.annotation.VisibleForTesting;



import com.android.settingslib.R;

import com.android.settingslib.media.flags.Flags;
@@ -98,7 +98,7 @@ public class PhoneMediaDevice extends MediaDevice { 
            case TYPE_USB_ACCESSORY:

                name =

                        inputRoutingEnabledAndIsDesktop(context)

                                ? context.getString(R.string.media_transfer_usb_audio_name)

                                ? routeInfo.getName()

                                : context.getString(R.string.media_transfer_wired_headphone_name);

                break;

            case TYPE_DOCK:

packages/SettingsLib/tests/robotests/src/com/android/settingslib/media/PhoneMediaDeviceTest.java

+58 −5
Original line number Diff line number Diff line
@@ -17,7 +17,9 @@ 
package com.android.settingslib.media;



import static android.media.MediaRoute2Info.TYPE_BUILTIN_SPEAKER;

import static android.media.MediaRoute2Info.TYPE_USB_ACCESSORY;

import static android.media.MediaRoute2Info.TYPE_USB_DEVICE;

import static android.media.MediaRoute2Info.TYPE_USB_HEADSET;

import static android.media.MediaRoute2Info.TYPE_WIRED_HEADPHONES;

import static android.media.MediaRoute2Info.TYPE_WIRED_HEADSET;
@@ -120,28 +122,79 @@ public class PhoneMediaDeviceTest { 


    @EnableFlags(Flags.FLAG_ENABLE_AUDIO_INPUT_DEVICE_ROUTING_AND_VOLUME_CONTROL)

    @Test

    public void getName_returnCorrectName_desktop() {

    public void getName_returnCorrectName_desktop_wiredHeadphones() {

        ShadowSystemProperties.override("ro.build.characteristics", "desktop");



        when(mInfo.getType()).thenReturn(TYPE_WIRED_HEADPHONES);

        // Even if the MediaRoute2Info reports a name, the default string should still be displayed,

        // since the MediaRoute2Info name is only used for USB devices.

        when(mInfo.getName()).thenReturn("WIRED_HEADPHONES");



        assertThat(mPhoneMediaDevice.getName())

                .isEqualTo(mContext.getString(R.string.media_transfer_headphone_name));

    }



    @EnableFlags(Flags.FLAG_ENABLE_AUDIO_INPUT_DEVICE_ROUTING_AND_VOLUME_CONTROL)

    @Test

    public void getName_returnCorrectName_desktop_wiredHeadset() {

        ShadowSystemProperties.override("ro.build.characteristics", "desktop");



        when(mInfo.getType()).thenReturn(TYPE_WIRED_HEADSET);

        // Even if the MediaRoute2Info reports a name, the default string should still be displayed,

        // since the MediaRoute2Info name is only used for USB devices.

        when(mInfo.getName()).thenReturn("WIRED_HEADSET");



        assertThat(mPhoneMediaDevice.getName())

                .isEqualTo(mContext.getString(R.string.media_transfer_headphone_name));

    }



    @EnableFlags(Flags.FLAG_ENABLE_AUDIO_INPUT_DEVICE_ROUTING_AND_VOLUME_CONTROL)

    @Test

    public void getName_returnCorrectName_desktop_usbDevice() {

        ShadowSystemProperties.override("ro.build.characteristics", "desktop");



        when(mInfo.getType()).thenReturn(TYPE_USB_DEVICE);

        final String mediaRoute2InfoName = "USB-Audio - My Device";

        when(mInfo.getName()).thenReturn(mediaRoute2InfoName);



        assertThat(mPhoneMediaDevice.getName())

                .isEqualTo(mContext.getString(R.string.media_transfer_usb_audio_name));

        assertThat(mPhoneMediaDevice.getName()).isEqualTo(mediaRoute2InfoName);

    }



    @EnableFlags(Flags.FLAG_ENABLE_AUDIO_INPUT_DEVICE_ROUTING_AND_VOLUME_CONTROL)

    @Test

    public void getName_returnCorrectName_desktop_usbHeadset() {

        ShadowSystemProperties.override("ro.build.characteristics", "desktop");



        when(mInfo.getType()).thenReturn(TYPE_USB_HEADSET);

        final String mediaRoute2InfoName = "USB-Audio - My Headset";

        when(mInfo.getName()).thenReturn(mediaRoute2InfoName);



        assertThat(mPhoneMediaDevice.getName()).isEqualTo(mediaRoute2InfoName);

    }



    @EnableFlags(Flags.FLAG_ENABLE_AUDIO_INPUT_DEVICE_ROUTING_AND_VOLUME_CONTROL)

    @Test

    public void getName_returnCorrectName_desktop_usbAccessory() {

        ShadowSystemProperties.override("ro.build.characteristics", "desktop");



        when(mInfo.getType()).thenReturn(TYPE_USB_ACCESSORY);

        final String mediaRoute2InfoName = "USB-Audio - My Accessory";

        when(mInfo.getName()).thenReturn(mediaRoute2InfoName);



        assertThat(mPhoneMediaDevice.getName()).isEqualTo(mediaRoute2InfoName);

    }



    @EnableFlags(Flags.FLAG_ENABLE_AUDIO_INPUT_DEVICE_ROUTING_AND_VOLUME_CONTROL)

    @Test

    public void getName_returnCorrectName_desktop_builtinSpeaker() {

        ShadowSystemProperties.override("ro.build.characteristics", "desktop");



        when(mInfo.getType()).thenReturn(TYPE_BUILTIN_SPEAKER);

        // Even if the MediaRoute2Info reports a name, the default string should still be displayed,

        // since the MediaRoute2Info name is only used for USB devices.

        when(mInfo.getName()).thenReturn("Phone");



        assertThat(mPhoneMediaDevice.getName())

                .isEqualTo(getMediaTransferThisDeviceName(mContext));

        assertThat(mPhoneMediaDevice.getName()).isEqualTo(getMediaTransferThisDeviceName(mContext));

    }



    @EnableFlags(Flags.FLAG_ENABLE_AUDIO_POLICIES_DEVICE_AND_BLUETOOTH_CONTROLLER)