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

Add UiEvents for cancellation reasons previously not covered

Browse Source 
Bug: 260852245
Test: statsd_testdrive
Change-Id: Ie976d9f826e7d53089fe65ba0735b40a79587f6f
Merged-In: Ie976d9f826e7d53089fe65ba0735b40a79587f6f
This commit is contained in:
Yuri Lin
2022-12-01 17:35:02 -05:00
parent 04088cff0e
commit df3fe37fd3
2 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
core/java/android/service/notification/NotificationListenerService.java
View File

@@ -258,6 +258,8 @@ public abstract class NotificationListenerService extends Service {
public static final int REASON_CLEAR_DATA = 21;
/** Notification was canceled due to an assistant adjustment update. */
public static final int REASON_ASSISTANT_CANCEL = 22;
// If adding a new notification cancellation reason, you must also add handling for it in
// NotificationCancelledEvent.fromCancelReason.
/**
* @hide

11
services/core/java/com/android/server/notification/NotificationRecordLogger.java
View File

@@ -18,8 +18,8 @@ package com.android.server.notification;
import static android.service.notification.NotificationListenerService.REASON_ASSISTANT_CANCEL;
import static android.service.notification.NotificationListenerService.REASON_CANCEL;
import static android.service.notification.NotificationListenerService.REASON_CLEAR_DATA;
import static android.service.notification.NotificationListenerService.REASON_CLICK;
import static android.service.notification.NotificationListenerService.REASON_TIMEOUT;
import android.annotation.NonNull;
import android.annotation.Nullable;
@@ -172,7 +172,12 @@ public interface NotificationRecordLogger {
NOTIFICATION_CANCEL_SNOOZED(181),
@UiEvent(doc = "Notification was canceled due to timeout")
NOTIFICATION_CANCEL_TIMEOUT(182),
// Values 183-189 reserved for future system dismissal reasons
@UiEvent(doc = "Notification was canceled due to the backing channel being deleted")
NOTIFICATION_CANCEL_CHANNEL_REMOVED(1261),
@UiEvent(doc = "Notification was canceled due to the app's storage being cleared")
NOTIFICATION_CANCEL_CLEAR_DATA(1262),
// Values above this line must remain in the same order as the corresponding
// NotificationCancelReason enum values.
@UiEvent(doc = "Notification was canceled due to user dismissal of a peeking notification.")
NOTIFICATION_CANCEL_USER_PEEK(190),
@UiEvent(doc = "Notification was canceled due to user dismissal from the always-on display")
@@ -208,7 +213,7 @@ public interface NotificationRecordLogger {
// Most cancel reasons do not have a meaningful surface. Reason codes map directly
// to NotificationCancelledEvent codes.
if (surface == NotificationStats.DISMISSAL_OTHER) {
if ((REASON_CLICK <= reason) && (reason <= REASON_TIMEOUT)) {
if ((REASON_CLICK <= reason) && (reason <= REASON_CLEAR_DATA)) {
return NotificationCancelledEvent.values()[reason];
}
if (reason == REASON_ASSISTANT_CANCEL) {