From 3de59f24a0dffadded90e7a15710231ff6622cf1 Mon Sep 17 00:00:00 2001 From: Sunny Goyal Date: Mon, 4 Dec 2023 18:53:47 +0000 Subject: [PATCH] Revert "Revert "Removing unnecessary iconFactory class when gene..." Revert submission 25518508-revert-25483550-themed-badge-GJNKEECDMN Reason for revert: Fixed badging bug Reverted changes: /q/submissionid:25518508-revert-25483550-themed-badge-GJNKEECDMN Change-Id: I0f378eaabcb7bdea855b80a33bdec007c9b11436 --- .../com/android/quickstep/TaskIconCache.java | 17 ++++++----------- src/com/android/launcher3/Utilities.java | 13 ++++++------- 2 files changed, 12 insertions(+), 18 deletions(-) diff --git a/quickstep/src/com/android/quickstep/TaskIconCache.java b/quickstep/src/com/android/quickstep/TaskIconCache.java index 20a751be7a..1b3f5989c4 100644 --- a/quickstep/src/com/android/quickstep/TaskIconCache.java +++ b/quickstep/src/com/android/quickstep/TaskIconCache.java @@ -31,7 +31,6 @@ import android.graphics.drawable.Drawable; import android.os.UserHandle; import android.text.TextUtils; import android.util.SparseArray; -import android.view.accessibility.AccessibilityManager; import androidx.annotation.WorkerThread; @@ -45,6 +44,7 @@ import com.android.launcher3.pm.UserCache; import com.android.launcher3.util.DisplayController; import com.android.launcher3.util.DisplayController.DisplayInfoChangeListener; import com.android.launcher3.util.DisplayController.Info; +import com.android.launcher3.util.FlagOp; import com.android.launcher3.util.Preconditions; import com.android.quickstep.util.CancellableTask; import com.android.quickstep.util.TaskKeyLruCache; @@ -62,7 +62,6 @@ import java.util.function.Consumer; public class TaskIconCache implements DisplayInfoChangeListener { private final Executor mBgExecutor; - private final AccessibilityManager mAccessibilityManager; private final Context mContext; private final TaskKeyLruCache mIconCache; @@ -79,7 +78,6 @@ public class TaskIconCache implements DisplayInfoChangeListener { public TaskIconCache(Context context, Executor bgExecutor, IconProvider iconProvider) { mContext = context; mBgExecutor = bgExecutor; - mAccessibilityManager = context.getSystemService(AccessibilityManager.class); mIconProvider = iconProvider; Resources res = context.getResources(); @@ -238,14 +236,11 @@ public class TaskIconCache implements DisplayInfoChangeListener { if ((index = mDefaultIcons.indexOfKey(userId)) >= 0) { return mDefaultIcons.valueAt(index).newIcon(mContext); } else { - try (BaseIconFactory li = getIconFactory()) { - BitmapInfo info = mDefaultIconBase.withFlags( - li.getBitmapFlagOp(new IconOptions() - .setUser(UserCache.INSTANCE.get(mContext) - .getUserInfo(UserHandle.of(userId))))); - mDefaultIcons.put(userId, info); - return info.newIcon(mContext); - } + BitmapInfo info = mDefaultIconBase.withFlags( + UserCache.INSTANCE.get(mContext).getUserInfo(UserHandle.of(userId)) + .applyBitmapInfoFlags(FlagOp.NO_OP)); + mDefaultIcons.put(userId, info); + return info.newIcon(mContext); } } } diff --git a/src/com/android/launcher3/Utilities.java b/src/com/android/launcher3/Utilities.java index 8ba6d2c53e..3f1ccb9bd2 100644 --- a/src/com/android/launcher3/Utilities.java +++ b/src/com/android/launcher3/Utilities.java @@ -79,7 +79,6 @@ import androidx.core.graphics.ColorUtils; import com.android.launcher3.dragndrop.FolderAdaptiveIcon; import com.android.launcher3.graphics.TintedDrawableSpan; -import com.android.launcher3.icons.BaseIconFactory; import com.android.launcher3.icons.BitmapInfo; import com.android.launcher3.icons.LauncherIcons; import com.android.launcher3.icons.ShortcutCachingLogic; @@ -91,6 +90,7 @@ import com.android.launcher3.pm.UserCache; import com.android.launcher3.shortcuts.ShortcutKey; import com.android.launcher3.shortcuts.ShortcutRequest; import com.android.launcher3.testing.shared.ResourceUtils; +import com.android.launcher3.util.FlagOp; import com.android.launcher3.util.IntArray; import com.android.launcher3.util.SplitConfigurationOptions.SplitPositionOption; import com.android.launcher3.util.Themes; @@ -676,12 +676,11 @@ public final class Utilities { } if (badge == null) { - try (LauncherIcons li = LauncherIcons.obtain(context)) { - badge = BitmapInfo.LOW_RES_INFO.withFlags( - li.getBitmapFlagOp(new BaseIconFactory.IconOptions().setUser( - UserCache.INSTANCE.get(context).getUserInfo(info.user)))) - .getBadgeDrawable(context, useTheme); - } + badge = BitmapInfo.LOW_RES_INFO.withFlags( + UserCache.INSTANCE.get(context) + .getUserInfo(info.user) + .applyBitmapInfoFlags(FlagOp.NO_OP)) + .getBadgeDrawable(context, useTheme); if (badge == null) { badge = new ColorDrawable(Color.TRANSPARENT); }