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

Commit d7fee325 authored by Michal Brzezinski's avatar Michal Brzezinski
Browse files

Removing logs for b/237743330 

The bug is about keyguard status bar and status bar being visible at the same time.
Bug doesn't have any new reports in the last 6-7 weeks so these bug-specific logs are no longer needed.

Bug: 237743330
Test: just removing logs
Change-Id: Ic25bbf46d23b154ba56b121dd0306c3cb83da032
parent 37b5e8b9

packages/SystemUI/src/com/android/keyguard/logging/KeyguardLogger.kt

+0 −29
Original line number Diff line number Diff line
@@ -54,35 +54,6 @@ class KeyguardLogger @Inject constructor(@KeyguardLog private val buffer: LogBuf 
        buffer.log(TAG, INFO, { str1 = arg.toString() }, { "$msg: $str1" })

    }



    // TODO: remove after b/237743330 is fixed

    fun logStatusBarCalculatedAlpha(alpha: Float) {

        buffer.log(TAG, DEBUG, { double1 = alpha.toDouble() }, { "Calculated new alpha: $double1" })

    }



    // TODO: remove after b/237743330 is fixed

    fun logStatusBarExplicitAlpha(alpha: Float) {

        buffer.log(

            TAG,

            DEBUG,

            { double1 = alpha.toDouble() },

            { "new mExplicitAlpha value: $double1" }

        )

    }



    // TODO: remove after b/237743330 is fixed

    fun logStatusBarAlphaVisibility(visibility: Int, alpha: Float, state: String) {

        buffer.log(

            TAG,

            DEBUG,

            {

                int1 = visibility

                double1 = alpha.toDouble()

                str1 = state

            },

            { "changing visibility to $int1 with alpha $double1 in state: $str1" }

        )

    }



    @JvmOverloads

    fun logBiometricMessage(

        @CompileTimeConstant context: String,

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

+0 −11
Original line number Diff line number Diff line
@@ -475,9 +475,6 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
            newAlpha = Math.min(getKeyguardContentsAlpha(), alphaQsExpansion)

                    * mKeyguardStatusBarAnimateAlpha

                    * (1.0f - mKeyguardHeadsUpShowingAmount);

            if (newAlpha != mView.getAlpha() && (newAlpha == 0 || newAlpha == 1)) {

                mLogger.logStatusBarCalculatedAlpha(newAlpha);

            }

        }



        boolean hideForBypass =
@@ -500,10 +497,6 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
        if (mDisableStateTracker.isDisabled()) {

            visibility = View.INVISIBLE;

        }

        if (visibility != mView.getVisibility()) {

            mLogger.logStatusBarAlphaVisibility(visibility, alpha,

                    StatusBarState.toString(mStatusBarState));

        }

        mView.setAlpha(alpha);

        mView.setVisibility(visibility);

    }
@@ -608,10 +601,6 @@ public class KeyguardStatusBarViewController extends ViewController<KeyguardStat 
     * @param alpha a value between 0 and 1. -1 if the value is to be reset/ignored.

     */

    public void setAlpha(float alpha) {

        if (mExplicitAlpha != alpha && (mExplicitAlpha == -1 || alpha == -1)) {

            // logged if value changed to ignored or from ignored

            mLogger.logStatusBarExplicitAlpha(alpha);

        }

        mExplicitAlpha = alpha;

        updateViewState();

    }