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

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

Fixed a crash with notification groups 

When updating a notification we were not checking if it is the summary
with children, but about the existence of the childrencontainer, which
can change and is never removed. This is now done correctly.

Change-Id: Ie3838613f414d7fcb158dc92c24e8939cfb7c2ea
Fixes: 29055879
parent 6ebba59a

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

+2 −4
Original line number Diff line number Diff line
@@ -18,9 +18,7 @@ package com.android.systemui.statusbar; 


import android.animation.Animator;

import android.animation.AnimatorListenerAdapter;

import android.animation.AnimatorSet;

import android.animation.ObjectAnimator;

import android.animation.ValueAnimator;

import android.animation.ValueAnimator.AnimatorUpdateListener;

import android.content.Context;

import android.graphics.drawable.AnimatedVectorDrawable;
@@ -1067,7 +1065,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView { 
            mIsSystemExpanded = expand;

            notifyHeightChanged(false /* needsAnimation */);

            logExpansionEvent(false, wasExpanded);

            if (mChildrenContainer != null) {

            if (mIsSummaryWithChildren) {

                mChildrenContainer.updateGroupOverflow();

            }

        }
@@ -1142,7 +1140,7 @@ public class ExpandableNotificationRow extends ActivatableNotificationView { 
    }



    public void updateChildrenHeaderAppearance() {

        if (mChildrenContainer != null) {

        if (mIsSummaryWithChildren) {

            mChildrenContainer.updateChildrenHeaderAppearance();

        }

    }