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

Commit e40025cc authored by Evan Laird's avatar Evan Laird
Browse files

Add wtf log if notification panel height is NaN 

NotificationPanelViewController#getMaxPanelHeightNonBypass() is the one
place where we aren't explicitly checking for NaN where it could be
introduced into PanelViewController mExpandedFraction.

Test: manual
Bug: 160688163
Change-Id: I3e5047295085cd61c3ceebe456c6560b28f616f7
parent f82f0bc2

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

+2 −2
Original line number Diff line number Diff line
@@ -1998,8 +1998,8 @@ public class NotificationPanelViewController extends PanelViewController { 
            maxHeight = calculatePanelHeightShade();

        }

        maxHeight = Math.max(min, maxHeight);

        if (maxHeight == 0) {

            Log.wtf(TAG, "maxPanelHeight is 0. getOverExpansionAmount(): "

        if (maxHeight == 0 || isNaN(maxHeight)) {

            Log.wtf(TAG, "maxPanelHeight is invalid. getOverExpansionAmount(): "

                    + getOverExpansionAmount() + ", calculatePanelHeightQsExpanded: "

                    + calculatePanelHeightQsExpanded() + ", calculatePanelHeightShade: "

                    + calculatePanelHeightShade() + ", mStatusBarMinHeight = "