Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5de8369e authored by Patrick Baumann's avatar Patrick Baumann
Browse files

Remove usages of Slog.wtf 

This change makes better use of logging in AppsFilter by logging warning
or info as appropriate.

Fixes: 156863453
Test: atest AppEnumerationTests
Change-Id: Icf4952f179aa63b29932d56414cb328a088adbe9
parent c12ddbbc

services/core/java/com/android/server/pm/AppsFilter.java

+3 −3
Original line number Diff line number Diff line
@@ -417,7 +417,7 @@ public class AppsFilter { 
    public void grantImplicitAccess(int recipientUid, int visibleUid) {

        if (recipientUid != visibleUid

                && mImplicitlyQueryable.add(recipientUid, visibleUid) && DEBUG_LOGGING) {

            Slog.wtf(TAG, "implicit access granted: " + recipientUid + " -> " + visibleUid);

            Slog.i(TAG, "implicit access granted: " + recipientUid + " -> " + visibleUid);

        }

    }
@@ -720,7 +720,7 @@ public class AppsFilter { 
                return false;

            }

            if (callingSetting == null) {

                Slog.wtf(TAG, "No setting found for non system uid " + callingUid);

                Slog.w(TAG, "No setting found for non system uid " + callingUid);

                return true;

            }

            final PackageSetting callingPkgSetting;
@@ -760,7 +760,7 @@ public class AppsFilter { 
            final AndroidPackage targetPkg = targetPkgSetting.pkg;

            if (targetPkg == null) {

                if (DEBUG_LOGGING) {

                    Slog.wtf(TAG, "shouldFilterApplication: " + "targetPkg is null");

                    Slog.w(TAG, "shouldFilterApplication: " + "targetPkg is null");

                }

                return true;

            }