From 47d2375255d85c66095050ae590afc91a5a1d7e5 Mon Sep 17 00:00:00 2001 From: Adam Lesinski Date: Mon, 11 Sep 2017 19:05:04 +0000 Subject: Revert "OMS: fix listener callbacks triggered by changes to target package" Some internal tests broke due to this. Revert for now to unblock, and investigate why this occurred. This reverts commit f08ad98acbb83abf7d30b6583bdfc188145a4215. Bug: 65319218 Change-Id: I1bfb2d4937708441eac2ee5909717abec10f4971 --- .../java/com/android/server/om/OverlayManagerServiceImpl.java | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/services/core/java/com/android/server/om/OverlayManagerServiceImpl.java b/services/core/java/com/android/server/om/OverlayManagerServiceImpl.java index c3957f432f4c..db6e9749535b 100644 --- a/services/core/java/com/android/server/om/OverlayManagerServiceImpl.java +++ b/services/core/java/com/android/server/om/OverlayManagerServiceImpl.java @@ -169,9 +169,8 @@ final class OverlayManagerServiceImpl { } final PackageInfo targetPackage = mPackageManager.getPackageInfo(packageName, userId); - if (updateAllOverlaysForTarget(packageName, userId, targetPackage)) { - mListener.onOverlaysChanged(packageName, userId); - } + updateAllOverlaysForTarget(packageName, userId, targetPackage); + mListener.onOverlaysChanged(packageName, userId); } void onTargetPackageChanged(@NonNull final String packageName, final int userId) { @@ -211,9 +210,7 @@ final class OverlayManagerServiceImpl { Slog.d(TAG, "onTargetPackageRemoved packageName=" + packageName + " userId=" + userId); } - if (updateAllOverlaysForTarget(packageName, userId, null)) { - mListener.onOverlaysChanged(packageName, userId); - } + updateAllOverlaysForTarget(packageName, userId, null); } /** -- cgit v1.2.3-59-g8ed1b