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

Commit bc384690 authored by Lucas Dupin's avatar Lucas Dupin
Browse files

Do not accept NaN as scrim values 

Scrim values are becoming NaN and causing the status bar window
to not collapse.
These are inavlid values and should not be accepted.

Bug: 150672579
Test: make
Change-Id: I8217a0fa9a08f94b98121d6584fafab1bca88b4d
parent 19282c39

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

+5 −0
Original line number Diff line number Diff line
@@ -16,6 +16,8 @@ 


package com.android.systemui.statusbar;



import static java.lang.Float.isNaN;



import android.annotation.NonNull;

import android.content.Context;

import android.graphics.Canvas;
@@ -179,6 +181,9 @@ public class ScrimView extends View { 
     * @param alpha Gradient alpha from 0 to 1.

     */

    public void setViewAlpha(float alpha) {

        if (isNaN(alpha)) {

            throw new IllegalArgumentException("alpha cannot be NaN: " + alpha);

        }

        if (alpha != mViewAlpha) {

            mViewAlpha = alpha;

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

+25 −1
Original line number Diff line number Diff line
@@ -16,6 +16,8 @@ 


package com.android.systemui.statusbar.phone;



import static java.lang.Float.isNaN;



import android.animation.Animator;

import android.animation.AnimatorListenerAdapter;

import android.animation.ValueAnimator;
@@ -289,6 +291,10 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, OnCo 
        mInFrontAlpha = state.getFrontAlpha();

        mBehindAlpha = state.getBehindAlpha();

        mBubbleAlpha = state.getBubbleAlpha();

        if (isNaN(mBehindAlpha) || isNaN(mInFrontAlpha)) {

            throw new IllegalStateException("Scrim opacity is NaN for state: " + state + ", front: "

                    + mInFrontAlpha + ", back: " + mBehindAlpha);

        }

        applyExpansionToAlpha();



        // Scrim might acquire focus when user is navigating with a D-pad or a keyboard.
@@ -416,6 +422,9 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, OnCo 
     * @param fraction From 0 to 1 where 0 means collapsed and 1 expanded.

     */

    public void setPanelExpansion(float fraction) {

        if (isNaN(fraction)) {

            throw new IllegalArgumentException("Fraction should not be NaN");

        }

        if (mExpansionFraction != fraction) {

            mExpansionFraction = fraction;
@@ -493,6 +502,10 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, OnCo 
            mBehindTint = ColorUtils.blendARGB(ScrimState.BOUNCER.getBehindTint(),

                    mState.getBehindTint(), interpolatedFract);

        }

        if (isNaN(mBehindAlpha) || isNaN(mInFrontAlpha)) {

            throw new IllegalStateException("Scrim opacity is NaN for state: " + mState

                    + ", front: " + mInFrontAlpha + ", back: " + mBehindAlpha);

        }

    }



    /**
@@ -548,6 +561,10 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, OnCo 
            float newBehindAlpha = mState.getBehindAlpha();

            if (mBehindAlpha != newBehindAlpha) {

                mBehindAlpha = newBehindAlpha;

                if (isNaN(mBehindAlpha)) {

                    throw new IllegalStateException("Scrim opacity is NaN for state: " + mState

                            + ", back: " + mBehindAlpha);

                }

                updateScrims();

            }

        }
@@ -950,6 +967,9 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, OnCo 


        pw.print("  mTracking=");

        pw.println(mTracking);



        pw.print("  mExpansionFraction=");

        pw.println(mExpansionFraction);

    }



    public void setWallpaperSupportsAmbientMode(boolean wallpaperSupportsAmbientMode) {
@@ -996,6 +1016,10 @@ public class ScrimController implements ViewTreeObserver.OnPreDrawListener, OnCo 
        // in this case, back-scrim needs to be re-evaluated

        if (mState == ScrimState.AOD || mState == ScrimState.PULSING) {

            float newBehindAlpha = mState.getBehindAlpha();

            if (isNaN(newBehindAlpha)) {

                throw new IllegalStateException("Scrim opacity is NaN for state: " + mState

                        + ", back: " + mBehindAlpha);

            }

            if (mBehindAlpha != newBehindAlpha) {

                mBehindAlpha = newBehindAlpha;

                updateScrims();