summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Adam Bookatz <bookatz@google.com> 2020-03-31 12:26:47 -0700
committer Adam Bookatz <bookatz@google.com> 2020-03-31 19:35:57 +0000
commit9b6d51e688f11d9b0c75d1cef8761589a87b2cac (patch)
tree61971afc3aa28a93c30020cd1682cb2ae0db2730
parentc70636175c92e9682469f3a56db2c1d1cbbc66fd (diff)
Avoid getUserBadgeNoBackgroundResId if no badge
Checks whether there actually is a badge before calling getUserBadgeNoBackgroundResId. (Otherwise a logcat error would have been logged, since it is only expected to be called if there is indeed a badge.) Test: manual confirmation that logcat error is gone Bug: 142482943 Change-Id: I635529d9c114b399e5626bdbb143aab29ae81a4b
-rw-r--r--core/java/android/app/ApplicationPackageManager.java5
1 files changed, 4 insertions, 1 deletions
diff --git a/core/java/android/app/ApplicationPackageManager.java b/core/java/android/app/ApplicationPackageManager.java
index a1ec27b3e9f7..f883b60b534f 100644
--- a/core/java/android/app/ApplicationPackageManager.java
+++ b/core/java/android/app/ApplicationPackageManager.java
@@ -1610,7 +1610,10 @@ public class ApplicationPackageManager extends PackageManager {
@Override
public Drawable getUserBadgeForDensityNoBackground(UserHandle user, int density) {
- Drawable badge = getProfileIconForDensity(user,
+ if (!hasUserBadge(user.getIdentifier())) {
+ return null;
+ }
+ Drawable badge = getDrawableForDensity(
getUserManager().getUserBadgeNoBackgroundResId(user.getIdentifier()), density);
if (badge != null) {
badge.setTint(getUserBadgeColor(user));