From 00a776682cd5662a28e204894f44dc4b1876b627 Mon Sep 17 00:00:00 2001 From: Hui Yu Date: Wed, 18 Mar 2020 10:25:55 -0700 Subject: [PATCH] Only check on launcher proc state when param visible is true. When launcher moves away from TOP, param visible is false, should skip the proc state check and move ahead to update AppOps. Bug: 151835293 Test: Manual test, open another app and move launcher away from top screen, observe AppOps is updated. Change-Id: I4b1562994d205d7e5b0327b95cf59e08b91ae048 --- .../android/server/appwidget/AppWidgetServiceImpl.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java b/services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java index ad21075809ecf..260703db190c9 100644 --- a/services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java +++ b/services/appwidget/java/com/android/server/appwidget/AppWidgetServiceImpl.java @@ -3647,10 +3647,12 @@ class AppWidgetServiceImpl extends IAppWidgetService.Stub implements WidgetBacku } private void updateAppOpsLocked(Host host, boolean visible) { - // The launcher must be at TOP. - final int procState = mActivityManagerInternal.getUidProcessState(host.id.uid); - if (procState > ActivityManager.PROCESS_STATE_TOP) { - return; + if (visible) { + final int procState = mActivityManagerInternal.getUidProcessState(host.id.uid); + if (procState > ActivityManager.PROCESS_STATE_TOP) { + // The launcher must be at TOP. + return; + } } final List allHomeCandidates = new ArrayList<>();