Revert "[PM] Add more logs to debug the flaky test"
This reverts commit 556114ba51ed32e043fd169806baf6e53ef498c8.
Reason for revert: Found the root cause, remove the logs
Change-Id: I53658b454cc9c6eaff53ff355fc221c689058df1
diff --git a/services/core/java/com/android/server/pm/ComputerEngine.java b/services/core/java/com/android/server/pm/ComputerEngine.java
index b228bb9..4665a72 100644
--- a/services/core/java/com/android/server/pm/ComputerEngine.java
+++ b/services/core/java/com/android/server/pm/ComputerEngine.java
@@ -5144,28 +5144,10 @@
}
updateOwnerPackageName = installSource.mUpdateOwnerPackageName;
-
- if (DEBUG_INSTALL) {
- Log.d(TAG, "ComputerEngine getInstallSourceInfo updateOwnerPackageName = "
- + updateOwnerPackageName + ", callingUid = " + callingUid + ", packageName = "
- + packageName + ", userId = " + userId);
- }
-
if (updateOwnerPackageName != null) {
final PackageStateInternal ps = mSettings.getPackage(updateOwnerPackageName);
final boolean isCallerSystemOrUpdateOwner = callingUid == Process.SYSTEM_UID
|| isCallerSameApp(updateOwnerPackageName, callingUid);
-
- if (DEBUG_INSTALL) {
- Log.d(TAG, "ComputerEngine getInstallSourceInfo ps = "
- + ps + ", isCallerSystemOrUpdateOwner =" + isCallerSystemOrUpdateOwner
- + ", isCallerSameApp = "
- + isCallerSameApp(updateOwnerPackageName, callingUid) + ", filter = "
- + shouldFilterApplicationIncludingUninstalled(ps, callingUid, userId)
- + ", FromManagedUserOrProfile = "
- + isCallerFromManagedUserOrProfile(userId));
- }
-
// Except for package visibility filtering, we also hide update owner if the installer
// is in the managed user or profile. As we don't enforce the update ownership for the
// managed user and profile, knowing there's an update owner is meaningless in that
@@ -5177,11 +5159,6 @@
}
}
- if (DEBUG_INSTALL) {
- Log.d(TAG, "ComputerEngine getInstallSourceInfo updateOwnerPackageName = "
- + updateOwnerPackageName);
- }
-
if (installSource.mIsInitiatingPackageUninstalled) {
// We can't check visibility in the usual way, since the initiating package is no
// longer present. So we apply simpler rules to whether to expose the info: