From d1af446feb66d4a095ff18a90c0c4c7ff1d57220 Mon Sep 17 00:00:00 2001 From: Julia Reynolds Date: Wed, 8 Jul 2015 11:12:28 -0400 Subject: [PATCH] Show a default icon for permissions that lack a group. Bug: 22229417 Change-Id: I82e12504adcca5d3fb6902109a3d7075a4739843 --- .../android/widget/AppSecurityPermissions.java | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/core/java/android/widget/AppSecurityPermissions.java b/core/java/android/widget/AppSecurityPermissions.java index c54b28d097df6..d1af9dc8a2552 100644 --- a/core/java/android/widget/AppSecurityPermissions.java +++ b/core/java/android/widget/AppSecurityPermissions.java @@ -98,18 +98,12 @@ public class AppSecurityPermissions { super(info); } - public Drawable loadGroupIcon(PackageManager pm) { + public Drawable loadGroupIcon(Context context, PackageManager pm) { if (icon != 0) { return loadUnbadgedIcon(pm); } else { - ApplicationInfo appInfo; - try { - appInfo = pm.getApplicationInfo(packageName, 0); - return appInfo.loadUnbadgedIcon(pm); - } catch (NameNotFoundException e) { - } + return context.getDrawable(R.drawable.ic_perm_device_info); } - return null; } } @@ -163,7 +157,7 @@ public class AppSecurityPermissions { PackageManager pm = getContext().getPackageManager(); Drawable icon = null; if (first) { - icon = grp.loadGroupIcon(pm); + icon = grp.loadGroupIcon(getContext(), pm); } CharSequence label = perm.mLabel; if (perm.mNew && newPermPrefix != null) { @@ -213,7 +207,7 @@ public class AppSecurityPermissions { builder.setMessage(sbuilder.toString()); } builder.setCancelable(true); - builder.setIcon(mGroup.loadGroupIcon(pm)); + builder.setIcon(mGroup.loadGroupIcon(getContext(), pm)); addRevokeUIIfNecessary(builder); mDialog = builder.show(); mDialog.setCanceledOnTouchOutside(true);