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

Commit 3eb8fb43 authored by Michael Mikhail's avatar Michael Mikhail
Browse files

Ensure UMO is added to current media 

Small fix, when apps post same notification for many times sometimes
active field is false and then converted to true in the next post.
Therefore, it is better to compare the size of current media not the
sorted media.

Flag: com.android.systemui.scene_container
Bug: 326408371
Test: atest SystemUiRoboTests:MediaFilterRepositoryTest
Change-Id: Ib05b4d35a3614976692330d9ff56d82469d7ec85
parent 03482623

packages/SystemUI/multivalentTests/src/com/android/systemui/media/controls/data/repository/MediaFilterRepositoryTest.kt

+0 −1
Original line number Diff line number Diff line
@@ -152,7 +152,6 @@ class MediaFilterRepositoryTest : SysuiTestCase() { 
            underTest.setRecommendation(mediaRecommendation.copy(isActive = false))



            assertThat(smartspaceMediaData).isNotEqualTo(mediaRecommendation)

            assertThat(smartspaceMediaData?.isActive).isFalse()

            assertThat(underTest.isRecommendationActive()).isFalse()

        }

packages/SystemUI/src/com/android/systemui/media/controls/data/repository/MediaFilterRepository.kt

+1 −1
Original line number Diff line number Diff line
@@ -218,7 +218,7 @@ constructor( 
                        mediaFromRecPackageName = null

                        _currentMedia.value = sortedMap.values.toList()

                    }

                } else if (sortedMap.size > sortedMedia.size && it.active) {

                } else if (sortedMap.size > _currentMedia.value.size && it.active) {

                    _currentMedia.value = sortedMap.values.toList()

                } else {

                    // When loading an update for an existing media control.