diff options
| author | 2020-03-31 12:26:47 -0700 | |
|---|---|---|
| committer | 2020-03-31 19:35:57 +0000 | |
| commit | 9b6d51e688f11d9b0c75d1cef8761589a87b2cac (patch) | |
| tree | 61971afc3aa28a93c30020cd1682cb2ae0db2730 | |
| parent | c70636175c92e9682469f3a56db2c1d1cbbc66fd (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.java | 5 |
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)); |