diff options
| author | 2022-12-12 17:40:54 +0000 | |
|---|---|---|
| committer | 2022-12-12 17:40:54 +0000 | |
| commit | d7fee325b7b28021e3f745ea385c50e1a7580860 (patch) | |
| tree | 6a5ffd5d3e1c8ae3f7d2a42d4693729f4ebf3a52 | |
| parent | 37b5e8b974544937ac07264671d1e044fa60a622 (diff) | |
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
| -rw-r--r-- | packages/SystemUI/src/com/android/keyguard/logging/KeyguardLogger.kt | 29 | ||||
| -rw-r--r-- | packages/SystemUI/src/com/android/systemui/statusbar/phone/KeyguardStatusBarViewController.java | 11 |
2 files changed, 0 insertions, 40 deletions
diff --git a/packages/SystemUI/src/com/android/keyguard/logging/KeyguardLogger.kt b/packages/SystemUI/src/com/android/keyguard/logging/KeyguardLogger.kt index 9e58500c7206..886d110de369 100644 --- a/packages/SystemUI/src/com/android/keyguard/logging/KeyguardLogger.kt +++ b/packages/SystemUI/src/com/android/keyguard/logging/KeyguardLogger.kt @@ -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, diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/phone/KeyguardStatusBarViewController.java b/packages/SystemUI/src/com/android/systemui/statusbar/phone/KeyguardStatusBarViewController.java index d4dc1dc197a5..348357445223 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/phone/KeyguardStatusBarViewController.java +++ b/packages/SystemUI/src/com/android/systemui/statusbar/phone/KeyguardStatusBarViewController.java @@ -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(); } |