From 5d09b2e358ed219d130792c0837391a60474c98a Mon Sep 17 00:00:00 2001 From: Sunny Goyal Date: Thu, 9 Jul 2020 12:21:40 -0700 Subject: [PATCH] Updating some comments and variables Change-Id: Ib3223c66e9ce187cebc6ce26a46b228f95787778 --- .../src/com/android/quickstep/TouchInteractionService.java | 4 ++-- src/com/android/launcher3/BaseDraggingActivity.java | 2 +- .../android/launcher3/graphics/LauncherPreviewRenderer.java | 4 ++-- .../android/launcher3/util/MainThreadInitializedObject.java | 2 +- src/com/android/launcher3/util/TraceHelper.java | 2 +- src/com/android/launcher3/views/BaseDragLayer.java | 4 ++-- 6 files changed, 9 insertions(+), 9 deletions(-) diff --git a/quickstep/recents_ui_overrides/src/com/android/quickstep/TouchInteractionService.java b/quickstep/recents_ui_overrides/src/com/android/quickstep/TouchInteractionService.java index df5e0d9912..a63939a1ac 100644 --- a/quickstep/recents_ui_overrides/src/com/android/quickstep/TouchInteractionService.java +++ b/quickstep/recents_ui_overrides/src/com/android/quickstep/TouchInteractionService.java @@ -546,7 +546,7 @@ public class TouchInteractionService extends Service implements PluginListener mAM.getRunningTask(false /* filterOnlyVisibleRecents */))); } return gestureState; @@ -659,7 +659,7 @@ public class TouchInteractionService extends Service implements PluginListener mAM.getRunningTask(true /* filterOnlyVisibleRecents */))); ComponentName homeComponent = mOverviewComponentObserver.getHomeIntent().getComponent(); ComponentName runningComponent = diff --git a/src/com/android/launcher3/BaseDraggingActivity.java b/src/com/android/launcher3/BaseDraggingActivity.java index 296fb7ae78..112126b988 100644 --- a/src/com/android/launcher3/BaseDraggingActivity.java +++ b/src/com/android/launcher3/BaseDraggingActivity.java @@ -81,7 +81,7 @@ public abstract class BaseDraggingActivity extends BaseActivity super.onCreate(savedInstanceState); - mIsSafeModeEnabled = TraceHelper.whitelistIpcs("isSafeMode", + mIsSafeModeEnabled = TraceHelper.allowIpcs("isSafeMode", () -> getPackageManager().isSafeMode()); DefaultDisplay.INSTANCE.get(this).addChangeListener(this); diff --git a/src/com/android/launcher3/graphics/LauncherPreviewRenderer.java b/src/com/android/launcher3/graphics/LauncherPreviewRenderer.java index 458ffa368d..a424f8458a 100644 --- a/src/com/android/launcher3/graphics/LauncherPreviewRenderer.java +++ b/src/com/android/launcher3/graphics/LauncherPreviewRenderer.java @@ -126,7 +126,7 @@ public class LauncherPreviewRenderer { */ public static class PreviewContext extends ContextWrapper { - private static final Set WHITELIST = new HashSet<>( + private final Set mAllowedObjects = new HashSet<>( Arrays.asList(UserCache.INSTANCE, InstallSessionHelper.INSTANCE, LauncherAppState.INSTANCE, InvariantDeviceProfile.INSTANCE, CustomWidgetManager.INSTANCE, PluginManagerWrapper.INSTANCE)); @@ -160,7 +160,7 @@ public class LauncherPreviewRenderer { */ public T getObject(MainThreadInitializedObject mainThreadInitializedObject, MainThreadInitializedObject.ObjectProvider provider) { - if (!WHITELIST.contains(mainThreadInitializedObject)) { + if (!mAllowedObjects.contains(mainThreadInitializedObject)) { throw new IllegalStateException("Leaking unknown objects"); } if (mainThreadInitializedObject == LauncherAppState.INSTANCE) { diff --git a/src/com/android/launcher3/util/MainThreadInitializedObject.java b/src/com/android/launcher3/util/MainThreadInitializedObject.java index fc9f8f7ec4..f6003dd7bf 100644 --- a/src/com/android/launcher3/util/MainThreadInitializedObject.java +++ b/src/com/android/launcher3/util/MainThreadInitializedObject.java @@ -46,7 +46,7 @@ public class MainThreadInitializedObject { if (mValue == null) { if (Looper.myLooper() == Looper.getMainLooper()) { - mValue = TraceHelper.whitelistIpcs("main.thread.object", + mValue = TraceHelper.allowIpcs("main.thread.object", () -> mProvider.get(context.getApplicationContext())); } else { try { diff --git a/src/com/android/launcher3/util/TraceHelper.java b/src/com/android/launcher3/util/TraceHelper.java index 168227db6e..c23df77db1 100644 --- a/src/com/android/launcher3/util/TraceHelper.java +++ b/src/com/android/launcher3/util/TraceHelper.java @@ -78,7 +78,7 @@ public class TraceHelper { * Temporarily ignore blocking binder calls for the duration of this {@link Supplier}. */ @MainThread - public static T whitelistIpcs(String rpcName, Supplier supplier) { + public static T allowIpcs(String rpcName, Supplier supplier) { Object traceToken = INSTANCE.beginSection(rpcName, FLAG_IGNORE_BINDERS); try { return supplier.get(); diff --git a/src/com/android/launcher3/views/BaseDragLayer.java b/src/com/android/launcher3/views/BaseDragLayer.java index 2c75c74827..f60c65593a 100644 --- a/src/com/android/launcher3/views/BaseDragLayer.java +++ b/src/com/android/launcher3/views/BaseDragLayer.java @@ -605,13 +605,13 @@ public abstract class BaseDragLayer */ private boolean computeAllowSysuiScrims(@Nullable WallpaperInfo newWallpaperInfo) { if (newWallpaperInfo == null) { - // New wallpaper is static, not live. Thus, blacklist isn't applicable. + // Static wallpapers need scrim unless determined otherwise by wallpaperColors. return true; } ComponentName newWallpaper = newWallpaperInfo.getComponent(); for (String wallpaperWithoutScrim : mWallpapersWithoutSysuiScrims) { if (newWallpaper.equals(ComponentName.unflattenFromString(wallpaperWithoutScrim))) { - // New wallpaper is blacklisted from showing a scrim. + // New wallpaper does not need a scrim. return false; } }