From ddd236f2c91b2af2f84957bf8797ab6cf182a07b Mon Sep 17 00:00:00 2001 From: Julia Reynolds Date: Wed, 7 Apr 2021 12:10:53 -0400 Subject: [PATCH] Add temp logging for zen policy changes Because by the time you notice the policy has changed the zen log has likely rolled over Test: manual in logcat Bug: 180205791 Change-Id: I766ce6609464e58f9a63c5be8eb5ddfc24bb4a5c --- .../core/java/com/android/server/notification/ZenLog.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/notification/ZenLog.java b/services/core/java/com/android/server/notification/ZenLog.java index f5d648910d909..1050835a98127 100644 --- a/services/core/java/com/android/server/notification/ZenLog.java +++ b/services/core/java/com/android/server/notification/ZenLog.java @@ -27,6 +27,7 @@ import android.service.notification.Condition; import android.service.notification.IConditionProvider; import android.service.notification.NotificationListenerService; import android.service.notification.ZenModeConfig; +import android.util.Log; import android.util.Slog; import java.io.PrintWriter; @@ -122,8 +123,11 @@ public class ZenLog { public static void traceSetNotificationPolicy(String pkg, int targetSdk, NotificationManager.Policy policy) { - append(TYPE_SET_NOTIFICATION_POLICY, "pkg=" + pkg + " targetSdk=" + targetSdk - + " NotificationPolicy=" + policy.toString()); + String policyLog = "pkg=" + pkg + " targetSdk=" + targetSdk + + " NotificationPolicy=" + policy.toString(); + append(TYPE_SET_NOTIFICATION_POLICY, policyLog); + // TODO(b/180205791): remove when we can better surface apps that are changing policy + Log.d(TAG, "Zen Policy Changed: " + policyLog); } public static void traceSubscribe(Uri uri, IConditionProvider provider, RemoteException e) {