Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

also log user-initiated notification collapse events

Browse Source 
Bug: 62787514
Test: runtest systemui-notification
Change-Id: I8e7e3bad5d2d5bbd0cebe2c5927e83b4ead4db1c
This commit is contained in:
Chris Wren
2017-09-14 10:55:55 -04:00
parent 786354a53f
commit f7342718c0
2 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
proto/src/metrics_constants.proto
View File

@@ -33,7 +33,7 @@ message MetricsEvent {
// The view became hidden.
TYPE_CLOSE = 2;
// The view switched to detail mode (most relevant for quick settings tiles)
// The view switched to detail mode (most relevant for quick settings tiles and notifications)
TYPE_DETAIL = 3;
// The view or control was activated.
@@ -70,6 +70,9 @@ message MetricsEvent {
// Type for APP_TRANSITION_REPORTED_DRAWN event: The activity was started with restoring from
// a bundle.
TYPE_TRANSITION_REPORTED_DRAWN_WITH_BUNDLE = 13;
// The view switched to summary mode (most relevant for notifications)
TYPE_COLLAPSE = 14;
}
// Types of alerts, as bit field values

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

@@ -756,10 +756,11 @@ public class NotificationManagerService extends SystemService {
if (r != null) {
r.stats.onExpansionChanged(userAction, expanded);
final long now = System.currentTimeMillis();
if (userAction && expanded) {
if (userAction) {
MetricsLogger.action(r.getLogMaker(now)
.setCategory(MetricsEvent.NOTIFICATION_ITEM)
.setType(MetricsEvent.TYPE_DETAIL));
.setType(expanded ? MetricsEvent.TYPE_DETAIL
: MetricsEvent.TYPE_COLLAPSE));
}
EventLogTags.writeNotificationExpansion(key,
userAction ? 1 : 0, expanded ? 1 : 0,