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

Commit f9c53b63 authored by Esteban Talavera's avatar Esteban Talavera
Browse files

Log notification direct reply to TRON 

Bug: 118375812
Test: runtest systemui-notification
Change-Id: I29de32a36c0cafb869c8ae79bcaee67beb68c5c2
parent e9f5e860

proto/src/metrics_constants/metrics_constants.proto

+8 −0
Original line number Diff line number Diff line
@@ -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.

services/core/java/com/android/server/notification/NotificationManagerService.java

+3 −0
Original line number Diff line number Diff line
@@ -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);

                }

            }