From 6fa02e95ba41578d8e8d16f8201f90ca37634183 Mon Sep 17 00:00:00 2001 From: Felipe Leme Date: Wed, 31 Aug 2016 08:47:50 -0700 Subject: [PATCH] Remove wtf log when optional apps are not found. Test: none Change-Id: I9f94c2ab47eb78d05754c85aafc13e1e238ead04 Fixes: 31177447 --- .../server/net/NetworkPolicyManagerService.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java index f3fc676fe2e61..d5da66be3860a 100644 --- a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java +++ b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java @@ -513,12 +513,13 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub { try { app = pm.getApplicationInfoAsUser(pkg, PackageManager.MATCH_SYSTEM_ONLY, userId); } catch (PackageManager.NameNotFoundException e) { - // Should not happen - Slog.wtf(TAG, "No ApplicationInfo for package " + pkg); + if (LOGD) Slog.d(TAG, "No ApplicationInfo for package " + pkg); + // Ignore it - some apps on allow-in-data-usage-save are optional. continue; } if (!app.isPrivilegedApp()) { - Slog.wtf(TAG, "pm.getApplicationInfoAsUser() returned non-privileged app: " + pkg); + Slog.e(TAG, "addDefaultRestrictBackgroundWhitelistUidsUL(): " + + "skipping non-privileged app " + pkg); continue; } final int uid = UserHandle.getUid(userId, app.uid); @@ -528,8 +529,9 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub { + "background whitelist. Revoked status: " + mRestrictBackgroundWhitelistRevokedUids.get(uid)); if (!mRestrictBackgroundWhitelistRevokedUids.get(uid)) { - Slog.i(TAG, "adding default package " + pkg + " (uid " + uid + " for user " - + userId + ") to restrict background whitelist"); + if (LOGD) + Slog.d(TAG, "adding default package " + pkg + " (uid " + uid + " for user " + + userId + ") to restrict background whitelist"); mRestrictBackgroundWhitelistUids.append(uid, true); changed = true; }