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

Commit 1d5d5025 authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "[Chipbar] Don't add null icon descriptions to the main description."... 

Merge "[Chipbar] Don't add null icon descriptions to the main description." into tm-qpr-dev am: 6fa49434

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



Change-Id: I422bb23c17648010ff23dc1476a7ee2f05ccad21
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents ef005378 6fa49434

packages/SystemUI/src/com/android/systemui/temporarydisplay/chipbar/ChipbarCoordinator.kt

+8 −5
Original line number Diff line number Diff line
@@ -156,11 +156,14 @@ open class ChipbarCoordinator @Inject constructor( 
        }



        // ---- Overall accessibility ----

        currentView.requireViewById<ViewGroup>(

                R.id.chipbar_inner

        ).contentDescription =

            "${newInfo.startIcon.icon.contentDescription.loadContentDescription(context)} " +

                "${newInfo.text.loadText(context)}"

        val iconDesc = newInfo.startIcon.icon.contentDescription

        val loadedIconDesc = if (iconDesc != null) {

            "${iconDesc.loadContentDescription(context)} "

        } else {

            ""

        }

        currentView.requireViewById<ViewGroup>(R.id.chipbar_inner).contentDescription =

            "$loadedIconDesc${newInfo.text.loadText(context)}"



        // ---- Haptics ----

        newInfo.vibrationEffect?.let {

packages/SystemUI/tests/src/com/android/systemui/temporarydisplay/chipbar/ChipbarCoordinatorTest.kt

+29 −0
Original line number Diff line number Diff line
@@ -109,6 +109,35 @@ class ChipbarCoordinatorTest : SysuiTestCase() { 
        underTest.start()

    }



    @Test

    fun displayView_contentDescription_iconHasDescription() {

        underTest.displayView(

            createChipbarInfo(

                Icon.Resource(R.drawable.ic_cake, ContentDescription.Loaded("loadedCD")),

                Text.Loaded("text"),

                endItem = null,

            )

        )



        val contentDescView = getChipbarView().requireViewById<ViewGroup>(R.id.chipbar_inner)

        assertThat(contentDescView.contentDescription.toString()).contains("loadedCD")

        assertThat(contentDescView.contentDescription.toString()).contains("text")

    }



    @Test

    fun displayView_contentDescription_iconHasNoDescription() {

        underTest.displayView(

            createChipbarInfo(

                Icon.Resource(R.drawable.ic_cake, contentDescription = null),

                Text.Loaded("text"),

                endItem = null,

            )

        )



        val contentDescView = getChipbarView().requireViewById<ViewGroup>(R.id.chipbar_inner)

        assertThat(contentDescView.contentDescription.toString()).isEqualTo("text")

    }



    @Test

    fun displayView_loadedIcon_correctlyRendered() {

        val drawable = context.getDrawable(R.drawable.ic_celebration)!!