Merge "Allow user to block individual apps from resuming" into rvc-qpr-dev am: 3743d5526a (865643a1) · Commits · e / os / android_frameworks_base

core/java/android/provider/Settings.java

+8 −0

Original line number	Diff line number	Diff line
		@@ -8928,6 +8928,14 @@ public final class Settings {
		*/
		public static final String MEDIA_CONTROLS_RESUME = "qs_media_resumption";

		/**
		* Controls which packages are blocked from persisting in media controls when resumption is
		* enabled. The list of packages is set by the user in the Settings app.
		* @see Settings.Secure#MEDIA_CONTROLS_RESUME
		* @hide
		*/
		public static final String MEDIA_CONTROLS_RESUME_BLOCKED = "qs_media_resumption_blocked";

		/**
		* Controls if window magnification is enabled.
		* @hide

packages/SettingsProvider/src/android/provider/settings/backup/SecureSettings.java

+1 −0

Original line number	Diff line number	Diff line
		@@ -165,6 +165,7 @@ public class SecureSettings {
		Settings.Secure.AWARE_TAP_PAUSE_TOUCH_COUNT,
		Settings.Secure.PEOPLE_STRIP,
		Settings.Secure.MEDIA_CONTROLS_RESUME,
		Settings.Secure.MEDIA_CONTROLS_RESUME_BLOCKED,
		Settings.Secure.ACCESSIBILITY_MAGNIFICATION_MODE,
		Settings.Secure.ACCESSIBILITY_BUTTON_TARGETS
		};

packages/SettingsProvider/src/android/provider/settings/validators/SecureSettingsValidators.java

+2 −0

Original line number	Diff line number	Diff line
		@@ -244,6 +244,8 @@ public class SecureSettingsValidators {
		VALIDATORS.put(Secure.TAP_GESTURE, BOOLEAN_VALIDATOR);
		VALIDATORS.put(Secure.PEOPLE_STRIP, BOOLEAN_VALIDATOR);
		VALIDATORS.put(Secure.MEDIA_CONTROLS_RESUME, BOOLEAN_VALIDATOR);
		VALIDATORS.put(Secure.MEDIA_CONTROLS_RESUME_BLOCKED,
		COLON_SEPARATED_PACKAGE_LIST_VALIDATOR);
		VALIDATORS.put(Secure.ACCESSIBILITY_MAGNIFICATION_MODE,
		new InclusiveIntegerRangeValidator(
		Secure.ACCESSIBILITY_MAGNIFICATION_MODE_FULLSCREEN,

packages/SystemUI/src/com/android/systemui/media/MediaDataManager.kt

+23 −1

Original line number	Diff line number	Diff line
		@@ -55,6 +55,8 @@ import java.io.PrintWriter
		import java.util.concurrent.Executor
		import javax.inject.Inject
		import javax.inject.Singleton
		import kotlin.collections.ArrayList
		import kotlin.collections.LinkedHashMap

		// URI fields to try loading album art from
		private val ART_URIS = arrayOf(
		@@ -102,6 +104,17 @@ class MediaDataManager(

		private val listeners: MutableSet<Listener> = mutableSetOf()
		private val mediaEntries: LinkedHashMap<String, MediaData> = LinkedHashMap()
		internal var appsBlockedFromResume: MutableSet<String> = Utils.getBlockedMediaApps(context)
		set(value) {
		// Update list
		appsBlockedFromResume.clear()
		appsBlockedFromResume.addAll(value)

		// Remove any existing resume players that are now blocked
		appsBlockedFromResume.forEach {
		removeAllForPackage(it)
		}
		}

		@Inject
		constructor(
		@@ -523,7 +536,8 @@ class MediaDataManager(
		fun onNotificationRemoved(key: String) {
		Assert.isMainThread()
		val removed = mediaEntries.remove(key)
		if (useMediaResumption && removed?.resumeAction != null) {
		if (useMediaResumption && removed?.resumeAction != null &&
		!isBlockedFromResume(removed?.packageName)) {
		Log.d(TAG, "Not removing $key because resumable")
		// Move to resume key (aka package name) if that key doesn't already exist.
		val resumeAction = getResumeMediaAction(removed.resumeAction!!)
		@@ -560,6 +574,13 @@ class MediaDataManager(
		}
		}

		private fun isBlockedFromResume(packageName: String?): Boolean {
		if (packageName == null) {
		return true
		}
		return appsBlockedFromResume.contains(packageName)
		}

		fun setMediaResumptionEnabled(isEnabled: Boolean) {
		if (useMediaResumption == isEnabled) {
		return
		@@ -602,6 +623,7 @@ class MediaDataManager(
		println("listeners: $listeners")
		println("mediaEntries: $mediaEntries")
		println("useMediaResumption: $useMediaResumption")
		println("appsBlockedFromResume: $appsBlockedFromResume")
		}
		}
		}

packages/SystemUI/src/com/android/systemui/media/MediaResumeListener.kt

+15 −3

Original line number	Diff line number	Diff line
		@@ -52,6 +52,7 @@ class MediaResumeListener @Inject constructor(

		private var useMediaResumption: Boolean = Utils.useMediaResumption(context)
		private val resumeComponents: ConcurrentLinkedQueue<ComponentName> = ConcurrentLinkedQueue()
		private var blockedApps: MutableSet<String> = Utils.getBlockedMediaApps(context)

		private lateinit var mediaDataManager: MediaDataManager

		@@ -114,6 +115,14 @@ class MediaResumeListener @Inject constructor(
		mediaDataManager.setMediaResumptionEnabled(useMediaResumption)
		}
		}, Settings.Secure.MEDIA_CONTROLS_RESUME)

		// Listen to changes in which apps are allowed to persist
		tunerService.addTunable(object : TunerService.Tunable {
		override fun onTuningChanged(key: String?, newValue: String?) {
		blockedApps = Utils.getBlockedMediaApps(context)
		mediaDataManager.appsBlockedFromResume = blockedApps
		}
		}, Settings.Secure.MEDIA_CONTROLS_RESUME_BLOCKED)
		}

		fun isResumptionEnabled() = useMediaResumption
		@@ -144,17 +153,20 @@ class MediaResumeListener @Inject constructor(
		}

		resumeComponents.forEach {
		if (!blockedApps.contains(it.packageName)) {
		val browser = ResumeMediaBrowser(context, mediaBrowserCallback, it)
		browser.findRecentMedia()
		}
		}
		}

		override fun onMediaDataLoaded(key: String, oldKey: String?, data: MediaData) {
		if (useMediaResumption) {
		// If this had been started from a resume state, disconnect now that it's live
		mediaBrowser?.disconnect()
		// If we don't have a resume action, check if we haven't already
		if (data.resumeAction == null && !data.hasCheckedForResume) {
		if (data.resumeAction == null && !data.hasCheckedForResume &&
		!blockedApps.contains(data.packageName)) {
		// TODO also check for a media button receiver intended for restarting (b/154127084)
		Log.d(TAG, "Checking for service component for " + data.packageName)
		val pm = context.packageManager