drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Log notification direct reply to TRON"

Browse Source
This commit is contained in:
Esteban Talavera 2018-11-15 18:05:06 +00:00 committed by Android (Google) Code Review
commit 7a5926bb46
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
proto/src/metrics_constants/metrics_constants.proto
View File

@ -844,6 +844,8 @@ message MetricsEvent {
// PACKAGE: App that posted the notification
// DETAIL: Notification is expanded by user.
// PACKAGE: App that posted the notification
// COLLAPSE: Notification is collapsed by user.
// PACKAGE: App that posted the notification
// DISMISS: Notification is dismissed.
// PACKAGE: App that posted the notification
// SUBTYPE: Dismiss reason from NotificationManagerService.java
@ -6596,6 +6598,12 @@ message MetricsEvent {
// OS: Q
NOTIFICATION_ZEN_MODE_OVERRIDING_APP = 1589;
// ACTION: User sent a direct reply
// PACKAGE: App that posted the notification
// CATEGORY: NOTIFICATION
// OS: Q
NOTIFICATION_DIRECT_REPLY_ACTION = 1590;
// ---- End Q Constants, all Q constants go above this line ----
// Add new aosp constants above this line.

3
services/core/java/com/android/server/notification/NotificationManagerService.java
View File

@ -885,6 +885,9 @@ public class NotificationManagerService extends SystemService {
NotificationRecord r = mNotificationsByKey.get(key);
if (r != null) {
r.recordDirectReplied();
mMetricsLogger.write(r.getLogMaker()
.setCategory(MetricsEvent.NOTIFICATION_DIRECT_REPLY_ACTION)
.setType(MetricsEvent.TYPE_ACTION));
reportUserInteraction(r);
}
}