From 52d9a561d254ed334a3c4409bb2e50817176bf61 Mon Sep 17 00:00:00 2001 From: Beth Thibodeau Date: Mon, 27 Apr 2020 12:45:29 -0400 Subject: [PATCH] Ensure media resumption controls have a title This is a requirement stated in go/sysui-media-resumption-requirements but was not actually being enforced. Bug: 154823185 Test: manual Change-Id: Iba10b37ff42733d8ef76faf14b90ed17877a1918 --- packages/SystemUI/src/com/android/systemui/qs/QSPanel.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java b/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java index c7ce1af0c1fb3..cb3d5116a9cd0 100644 --- a/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java +++ b/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java @@ -343,6 +343,11 @@ public class QSPanel extends LinearLayout implements Tunable, Callback, Brightne return; } + if (desc == null || desc.getTitle() == null) { + Log.e(TAG, "Description incomplete"); + return; + } + Log.d(TAG, "adding track from browser: " + desc + ", " + component); QSMediaPlayer player = new QSMediaPlayer(mContext, QSPanel.this, null, mForegroundExecutor, mBackgroundExecutor, mActivityStarter);