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

Commit c3cd06a1 authored by Ibrahim Yilmaz's avatar Ibrahim Yilmaz
Browse files

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
parent d528be42

packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationContentView.java

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

                        }

                    }

                }