From 4db243a3db0879bb9734e6e3dea1fd661c8074b9 Mon Sep 17 00:00:00 2001 From: Howard Ro Date: Tue, 7 Aug 2018 15:44:25 -0700 Subject: Add a logging for ACTION_BAN_APP_NOTES Bug: 111215354 Test: adb logcat -b events Change-Id: I4e26adac7a7ad5053b919605b8a25e18bd93cac7 Merged-In: I4e26adac7a7ad5053b919605b8a25e18bd93cac7 --- proto/src/metrics_constants.proto | 5 +++-- .../com/android/server/notification/NotificationManagerService.java | 4 ++++ 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/proto/src/metrics_constants.proto b/proto/src/metrics_constants.proto index 143fa8634333..835f93590558 100644 --- a/proto/src/metrics_constants.proto +++ b/proto/src/metrics_constants.proto @@ -941,9 +941,10 @@ message MetricsEvent { // OS: 6.0 NOTIFICATION_ZEN_MODE_EVENT_RULE = 146; - // ACTION: App notification settings > Block Notifications + // ACTION: App notification settings > Block Notifications or long press on + // notification blocks. // CATEGORY: SETTINGS - // OS: 6.0 + // OS: 9.0 ACTION_BAN_APP_NOTES = 147; // ACTION: Notification shade > Dismiss all button diff --git a/services/core/java/com/android/server/notification/NotificationManagerService.java b/services/core/java/com/android/server/notification/NotificationManagerService.java index 3ed8e2e08e5b..8304e2691979 100644 --- a/services/core/java/com/android/server/notification/NotificationManagerService.java +++ b/services/core/java/com/android/server/notification/NotificationManagerService.java @@ -2119,6 +2119,10 @@ public class NotificationManagerService extends SystemService { enforceSystemOrSystemUI("setNotificationsEnabledForPackage"); mRankingHelper.setEnabled(pkg, uid, enabled); + mMetricsLogger.write(new LogMaker(MetricsEvent.ACTION_BAN_APP_NOTES) + .setType(MetricsEvent.TYPE_ACTION) + .setPackageName(pkg) + .setSubtype(enabled ? 1 : 0)); // Now, cancel any outstanding notifications that are part of a just-disabled app if (!enabled) { cancelAllNotificationsInt(MY_UID, MY_PID, pkg, null, 0, 0, true, -- cgit v1.2.3-59-g8ed1b