commit 729919670f36ea3c8f202779e25bd3352bf191c8
author Gustav Sennton <gsennton@google.com> Mon Jan 28 21:09:54 2019 +0000
committer Gustav Sennton <gsennton@google.com> Tue Jan 29 13:30:17 2019 +0000
tree 90e8e71413f0c8bf7bbee7461a3c6806ac54915f
parent 9b23e4f42baf2e545b30006b23dd216580c4cddd

Log notification location when a notification action is clicked.

Bug: 120767764
Test: press a smart action from notification shade, and one from a
heads-up notification to ensure the correct notification locations are
logged.
Change-Id: I2fb10bec9c0c1da0807581a0337f8f6b1adc620e

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

diff --git a/services/core/java/com/android/server/notification/NotificationManagerService.java b/services/core/java/com/android/server/notification/NotificationManagerService.java
index de3f50a..a1eeb70 100644
--- a/services/core/java/com/android/server/notification/NotificationManagerService.java
+++ b/services/core/java/com/android/server/notification/NotificationManagerService.java
@@ -769,7 +769,10 @@
                                 action.isContextual() ? 1 : 0)
                         .addTaggedData(
                                 MetricsEvent.NOTIFICATION_SMART_SUGGESTION_ASSISTANT_GENERATED,
-                                generatedByAssistant ? 1 : 0));
+                                generatedByAssistant ? 1 : 0)
+                        .addTaggedData(MetricsEvent.NOTIFICATION_LOCATION,
+                                nv.location.toMetricsEventEnum()));
+
                 EventLogTags.writeNotificationActionClicked(key, actionIndex,
                         r.getLifespanMs(now), r.getFreshnessMs(now), r.getExposureMs(now),
                         nv.rank, nv.count);