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

Commit c7652791 authored by Selim Cinek's avatar Selim Cinek
Browse files

Fixed a crash when a notification is updated 

Under certain conditions, the minheight could be
queried while reset or when an inflation error happened
for an existing notification.
Since all notifications have collapsed layout anyway
that will always be set after reset, we don't have
to null those out.

Test: click clear all, have first notification update during the animation.
Bug: 32333389
Fixes: 33961825
Change-Id: I14a75cd628726c8acd892a81b95de40f92275bb1
parent fb6ee6d6

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

+0 −5
Original line number Diff line number Diff line
@@ -293,10 +293,6 @@ public class NotificationContentView extends FrameLayout { 
    }



    public void reset() {

        if (mContractedChild != null) {

            mContractedChild.animate().cancel();

            removeView(mContractedChild);

        }

        mPreviousExpandedRemoteInputIntent = null;

        if (mExpandedRemoteInput != null) {

            mExpandedRemoteInput.onNotificationUpdateOrReset();
@@ -327,7 +323,6 @@ public class NotificationContentView extends FrameLayout { 
            removeView(mHeadsUpChild);

            mHeadsUpRemoteInput = null;

        }

        mContractedChild = null;

        mExpandedChild = null;

        mHeadsUpChild = null;

    }