Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Check ongoing flag for media control dismissal" into tm-qpr-dev

Browse Source
This commit is contained in:
Beth Thibodeau
2023-01-13 20:48:28 +00:00
committed by Android (Google) Code Review
parent fa87b00c96 a9a94a185b
commit f5fb9c6346
2 changed files with 35 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/SystemUI/src/com/android/systemui/media/controls/pipeline/MediaDataManager.kt
View File

@@ -863,7 +863,7 @@ class MediaDataManager(
notificationKey = key,
hasCheckedForResume = hasCheckedForResume,
isPlaying = isPlaying,
isClearable = sbn.isClearable(),
isClearable = !sbn.isOngoing,
lastActive = lastActive,
instanceId = instanceId,
appUid = appUid,

34
packages/SystemUI/tests/src/com/android/systemui/media/controls/pipeline/MediaDataManagerTest.kt
View File

@@ -17,6 +17,7 @@
package com.android.systemui.media.controls.pipeline
import android.app.Notification
import android.app.Notification.FLAG_NO_CLEAR
import android.app.Notification.MediaStyle
import android.app.PendingIntent
import android.app.smartspace.SmartspaceAction
@@ -1451,6 +1452,39 @@ class MediaDataManagerTest : SysuiTestCase() {
assertThat(mediaDataCaptor.value.semanticActions).isNull()
}
@Test
fun testNoClearNotOngoing_canDismiss() {
mediaNotification =
SbnBuilder().run {
setPkg(PACKAGE_NAME)
modifyNotification(context).also {
it.setSmallIcon(android.R.drawable.ic_media_pause)
it.setStyle(MediaStyle().apply { setMediaSession(session.sessionToken) })
it.setOngoing(false)
it.setFlag(FLAG_NO_CLEAR, true)
}
build()
}
addNotificationAndLoad()
assertThat(mediaDataCaptor.value.isClearable).isTrue()
}
@Test
fun testOngoing_cannotDismiss() {
mediaNotification =
SbnBuilder().run {
setPkg(PACKAGE_NAME)
modifyNotification(context).also {
it.setSmallIcon(android.R.drawable.ic_media_pause)
it.setStyle(MediaStyle().apply { setMediaSession(session.sessionToken) })
it.setOngoing(true)
}
build()
}
addNotificationAndLoad()
assertThat(mediaDataCaptor.value.isClearable).isFalse()
}
/** Helper function to add a media notification and capture the resulting MediaData */
private fun addNotificationAndLoad() {
mediaDataManager.onNotificationAdded(KEY, mediaNotification)