diff options
| author | 2023-10-13 22:43:06 +0000 | |
|---|---|---|
| committer | 2023-10-13 22:43:06 +0000 | |
| commit | c3cd06a1010e9c9b508bb5d9900c4ea2ddcd559e (patch) | |
| tree | e623f4775cf4ee708e3d6d0cde04d497d949df58 | |
| parent | d528be42c187f3c821f7fc7c52a251e2afbeb4de (diff) | |
Focus and Close over Contoller instead of View
applyRemoteInput might trigger focus/close events of the RemoteInputView. This CL makes these operations over controller instead of View.
This will help us generate better logs in ag/25032656.
Bug: 290068526
Test: SystemUITests and Notification update while replying should work as is.
Change-Id: I520096884b5d782241de461d8a724114dbc9248e
| -rw-r--r-- | packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationContentView.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationContentView.java b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationContentView.java index a27a305428c4..60e75ff9e6e1 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationContentView.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationContentView.java @@ -1363,12 +1363,12 @@ public class NotificationContentView extends FrameLayout implements Notification result.mController.setPendingIntent(existingPendingIntent); } if (result.mController.updatePendingIntentFromActions(actions)) { - if (!result.mView.isActive()) { - result.mView.focus(); + if (!result.mController.isActive()) { + result.mController.focus(); } } else { - if (result.mView.isActive()) { - result.mView.close(); + if (result.mController.isActive()) { + result.mController.close(); } } } |