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

Commit 635798b2 authored by Shaowei Shen's avatar Shaowei Shen Committed by Automerger Merge Worker
Browse files

Merge "[Output Swithcer] Fix volume control issue for Group" into tm-dev am: c3b88dbe 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/19565636



Change-Id: I1648ea98c757f6d3ca9043de4ad9f493d7fd5607
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 8f6591ca c3b88dbe

packages/SystemUI/src/com/android/systemui/media/dialog/MediaOutputAdapter.java

+1 −0
Original line number Diff line number Diff line
@@ -269,6 +269,7 @@ public class MediaOutputAdapter extends MediaOutputBaseAdapter { 
        }



        private void onGroupActionTriggered(boolean isChecked, MediaDevice device) {

            disableSeekBar();

            if (isChecked && isDeviceIncluded(mController.getSelectableMediaDevice(), device)) {

                mController.addDeviceToPlayMedia(device);

            } else if (!isChecked && isDeviceIncluded(mController.getDeselectableMediaDevice(),

packages/SystemUI/src/com/android/systemui/media/dialog/MediaOutputBaseAdapter.java

+8 −1
Original line number Diff line number Diff line
@@ -273,6 +273,8 @@ public abstract class MediaOutputBaseAdapter extends 
        void initSeekbar(MediaDevice device, boolean isCurrentSeekbarInvisible) {

            if (!mController.isVolumeControlEnabled(device)) {

                disableSeekBar();

            } else {

                enableSeekBar();

            }

            mSeekBar.setMaxVolume(device.getMaxVolume());

            final int currentVolume = device.getCurrentVolume();
@@ -417,11 +419,16 @@ public abstract class MediaOutputBaseAdapter extends 
            return drawable;

        }



        private void disableSeekBar() {

        protected void disableSeekBar() {

            mSeekBar.setEnabled(false);

            mSeekBar.setOnTouchListener((v, event) -> true);

        }



        private void enableSeekBar() {

            mSeekBar.setEnabled(true);

            mSeekBar.setOnTouchListener((v, event) -> false);

        }



        protected void setUpDeviceIcon(MediaDevice device) {

            ThreadUtils.postOnBackgroundThread(() -> {

                Icon icon = mController.getDeviceIconCompat(device).toIcon(mContext);

packages/SystemUI/src/com/android/systemui/media/dialog/MediaOutputMetricLogger.java

+8 −0
Original line number Diff line number Diff line
@@ -194,6 +194,11 @@ public class MediaOutputMetricLogger { 
    }



    private int getLoggingDeviceType(MediaDevice device, boolean isSourceDevice) {

        if (device == null) {

            return isSourceDevice

                    ? SysUiStatsLog.MEDIA_OUTPUT_OP_SWITCH_REPORTED__SOURCE__UNKNOWN_TYPE

                    : SysUiStatsLog.MEDIA_OUTPUT_OP_SWITCH_REPORTED__TARGET__UNKNOWN_TYPE;

        }

        switch (device.getDeviceType()) {

            case MediaDevice.MediaDeviceType.TYPE_PHONE_DEVICE:

                return isSourceDevice
@@ -229,6 +234,9 @@ public class MediaOutputMetricLogger { 
    }



    private int getInteractionDeviceType(MediaDevice device) {

        if (device == null) {

            return SysUiStatsLog.MEDIA_OUTPUT_OP_INTERACTION_REPORTED__TARGET__UNKNOWN_TYPE;

        }

        switch (device.getDeviceType()) {

            case MediaDevice.MediaDeviceType.TYPE_PHONE_DEVICE:

                return SysUiStatsLog.MEDIA_OUTPUT_OP_INTERACTION_REPORTED__TARGET__BUILTIN_SPEAKER;

packages/SystemUI/tests/src/com/android/systemui/media/dialog/MediaOutputAdapterTest.java

+26 −0
Original line number Diff line number Diff line
@@ -274,4 +274,30 @@ public class MediaOutputAdapterTest extends SysuiTestCase { 


        verify(mMediaOutputController).connectDevice(mMediaDevice2);

    }



    @Test

    public void onItemClick_onGroupActionTriggered_verifySeekbarDisabled() {

        when(mMediaOutputController.getSelectedMediaDevice()).thenReturn(mMediaDevices);

        List<MediaDevice> selectableDevices = new ArrayList<>();

        selectableDevices.add(mMediaDevice1);

        when(mMediaOutputController.getSelectableMediaDevice()).thenReturn(selectableDevices);

        when(mMediaOutputController.hasAdjustVolumeUserRestriction()).thenReturn(true);

        mMediaOutputAdapter.onBindViewHolder(mViewHolder, 0);



        mViewHolder.mContainerLayout.performClick();



        assertThat(mViewHolder.mSeekBar.isEnabled()).isFalse();

    }



    @Test

    public void onBindViewHolder_volumeControlChangeToEnabled_enableSeekbarAgain() {

        when(mMediaOutputController.isVolumeControlEnabled(mMediaDevice1)).thenReturn(false);

        mMediaOutputAdapter.onBindViewHolder(mViewHolder, 0);

        assertThat(mViewHolder.mSeekBar.isEnabled()).isFalse();



        when(mMediaOutputController.isVolumeControlEnabled(mMediaDevice1)).thenReturn(true);

        mMediaOutputAdapter.onBindViewHolder(mViewHolder, 0);



        assertThat(mViewHolder.mSeekBar.isEnabled()).isTrue();

    }

}