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

Commit 778dd609 authored by Ibrahim Yilmaz's avatar Ibrahim Yilmaz
Browse files

[Inline Reply] Trigger expanded visible listener 

Bug: 417719135
Test: Post a notification with inline reply on AOD and click reply. Observe Shade is opened after unlock and inline reply is activated.
Flag: EXEMPT Bug fix
Change-Id: Ia3135ed6db5ff4aaaf8efe3ef9e9e90b56306364
parent a6e6daf4

packages/SystemUI/multivalentTests/src/com/android/systemui/statusbar/phone/StatusBarRemoteInputCallbackTest.java

+4 −4
Original line number Diff line number Diff line
@@ -252,8 +252,8 @@ public class StatusBarRemoteInputCallbackTest extends SysuiTestCase { 
        } else {

            verify(mGroupExpansionManager).toggleGroupExpansion(enrEntry);

        }

        verify(privateLayout).setOnExpandedVisibleListener(any());

        verify(enr, never()).setUserExpanded(anyBoolean());

        verify(privateLayout, never()).setOnExpandedVisibleListener(any());

    }



    @Test
@@ -281,10 +281,10 @@ public class StatusBarRemoteInputCallbackTest extends SysuiTestCase { 
                enr, mock(View.class), false, onExpandedVisibleRunner);



        // THEN

        verify(privateLayout).setOnExpandedVisibleListener(any());

        verify(mGroupExpansionManager, never()).toggleGroupExpansion(enrEntry);

        verify(mGroupExpansionManager, never()).toggleGroupExpansion(any(NotificationEntryAdapter.class));

        verify(enr, never()).setUserExpanded(anyBoolean());

        verify(privateLayout, never()).setOnExpandedVisibleListener(any());

    }



    @Test
@@ -347,7 +347,7 @@ public class StatusBarRemoteInputCallbackTest extends SysuiTestCase { 


        // THEN

        verify(enr, never()).toggleExpansionState();

        verify(privateLayout, never()).setOnExpandedVisibleListener(onExpandedVisibleRunner);

        verify(privateLayout).setOnExpandedVisibleListener(onExpandedVisibleRunner);

        verify(enr, never()).setUserExpanded(anyBoolean());

        verify(mGroupExpansionManager, never()).toggleGroupExpansion(any(NotificationEntry.class));

        verify(mGroupExpansionManager, never()).toggleGroupExpansion(any(NotificationEntryAdapter.class));
@@ -413,7 +413,7 @@ public class StatusBarRemoteInputCallbackTest extends SysuiTestCase { 


        // THEN

        verify(enr, never()).toggleExpansionState();

        verify(privateLayout, never()).setOnExpandedVisibleListener(onExpandedVisibleRunner);

        verify(privateLayout).setOnExpandedVisibleListener(onExpandedVisibleRunner);

        verify(enr, never()).setUserExpanded(anyBoolean());

        verify(mGroupExpansionManager, never()).toggleGroupExpansion(any(NotificationEntry.class));

        verify(mGroupExpansionManager, never()).toggleGroupExpansion(any(NotificationEntryAdapter.class));

packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarRemoteInputCallback.java

+1 −1
Original line number Diff line number Diff line
@@ -233,9 +233,9 @@ public class StatusBarRemoteInputCallback implements Callback, Callbacks, 
                    if (expandNotification) {

                        // notification isn't expanded, let's make sure it's expanded!

                        row.toggleExpansionState();

                        row.getPrivateLayout().setOnExpandedVisibleListener(runnable);

                    }

                }

                row.getPrivateLayout().setOnExpandedVisibleListener(runnable);

            } else {

                if (row.isChildInGroup() && !row.areChildrenExpanded()) {

                    // The group isn't expanded, let's make sure it's visible!