diff options
author | Svetoslav Ganov <svetoslavganov@google.com> | 2016-03-01 14:23:07 -0800 |
---|---|---|
committer | Svetoslav Ganov <svetoslavganov@google.com> | 2016-03-01 14:23:13 -0800 |
commit | b2016f5e06f77eeaa3962dee0b71d82b1ee56f01 (patch) | |
tree | 6aacfbea17d840431ed550d3cdfefcb08c359b8c /src | |
parent | dccf863b897856c37aa26c932e78b9b27808e092 (diff) | |
download | android_packages_apps_PackageInstaller-b2016f5e06f77eeaa3962dee0b71d82b1ee56f01.tar.gz android_packages_apps_PackageInstaller-b2016f5e06f77eeaa3962dee0b71d82b1ee56f01.tar.bz2 android_packages_apps_PackageInstaller-b2016f5e06f77eeaa3962dee0b71d82b1ee56f01.zip |
Use safe label in permission request UI
bug:23531798
Change-Id: I7cde3b9987c0e59b9aa148a5a3a597eec55a20a4
Diffstat (limited to 'src')
-rw-r--r-- | src/com/android/packageinstaller/permission/model/AppPermissions.java | 26 |
1 files changed, 3 insertions, 23 deletions
diff --git a/src/com/android/packageinstaller/permission/model/AppPermissions.java b/src/com/android/packageinstaller/permission/model/AppPermissions.java index ca28ab41..e455ef13 100644 --- a/src/com/android/packageinstaller/permission/model/AppPermissions.java +++ b/src/com/android/packageinstaller/permission/model/AppPermissions.java @@ -32,16 +32,6 @@ import java.util.LinkedHashMap; import java.util.List; public final class AppPermissions { - private static final float MAX_APP_LABEL_LENGTH_PIXELS = 500; - - private static final TextPaint sAppLabelEllipsizePaint = new TextPaint(); - static { - sAppLabelEllipsizePaint.setAntiAlias(true); - // Both text size and width are given in absolute pixels, for consistent truncation - // across devices; this value corresponds to the default 14dip size on an xdhpi device. - sAppLabelEllipsizePaint.setTextSize(42); - } - private final ArrayList<AppPermissionGroup> mGroups = new ArrayList<>(); private final LinkedHashMap<String, AppPermissionGroup> mNameToGroupMap = new LinkedHashMap<>(); @@ -63,7 +53,9 @@ public final class AppPermissions { mContext = context; mPackageInfo = packageInfo; mFilterPermissions = permissions; - mAppLabel = loadEllipsizedAppLabel(context, packageInfo); + mAppLabel = BidiFormatter.getInstance().unicodeWrap( + packageInfo.applicationInfo.loadSafeLabel( + context.getPackageManager()).toString()); mSortGroups = sortGroups; mOnErrorCallback = onErrorCallback; loadPermissionGroups(); @@ -177,16 +169,4 @@ public final class AppPermissions { } return false; } - - private static CharSequence loadEllipsizedAppLabel(Context context, PackageInfo packageInfo) { - String label = packageInfo.applicationInfo.loadLabel( - context.getPackageManager()).toString(); - String ellipsizedLabel = label.replace("\n", " "); - if (!DeviceUtils.isWear(context)) { - // Only ellipsize for non-Wear devices. - ellipsizedLabel = TextUtils.ellipsize(ellipsizedLabel, sAppLabelEllipsizePaint, - MAX_APP_LABEL_LENGTH_PIXELS, TextUtils.TruncateAt.END).toString(); - } - return BidiFormatter.getInstance().unicodeWrap(ellipsizedLabel); - } } |