Merge "Make phone call row in Privacy Dialog not clickable" into sc-dev

This commit is contained in:
TreeHugger Robot
2021-04-12 04:59:46 +00:00
committed by Android (Google) Code Review
3 changed files with 28 additions and 1 deletions

View File

@@ -51,6 +51,7 @@
/>
<ImageView
android:id="@+id/chevron"
android:layout_height="24dp"
android:layout_width="24dp"
android:layout_gravity="center_vertical"

View File

@@ -128,9 +128,14 @@ class PrivacyDialog(
)
} ?: firstLine
newView.requireViewById<TextView>(R.id.text).text = finalText
if (element.phoneCall) {
newView.requireViewById<View>(R.id.chevron).visibility = View.GONE
}
newView.apply {
setTag(element)
setOnClickListener(clickListener)
if (!element.phoneCall) {
setOnClickListener(clickListener)
}
}
return newView
}

View File

@@ -233,6 +233,27 @@ class PrivacyDialogTest : SysuiTestCase() {
)
}
@Test
fun testPhoneCallNotClickable() {
val element = PrivacyDialog.PrivacyElement(
PrivacyType.TYPE_MICROPHONE,
TEST_PACKAGE_NAME,
TEST_USER_ID,
"App",
null,
0L,
false,
false,
true
)
val list = listOf(element)
dialog = PrivacyDialog(context, list, starter)
dialog.show()
assertThat(dialog.requireViewById<View>(R.id.privacy_item).isClickable).isFalse()
assertThat(dialog.requireViewById<View>(R.id.chevron).visibility).isEqualTo(View.GONE)
}
@Test
fun testAttribution() {
val element = PrivacyDialog.PrivacyElement(