Add some debug logging for SC text fade animations.
Bug: 283953501
Test: treehugger
Relnote: N/a
Change-Id: Ic88c0d3be54861002e7ca381fbdf9fa72d95ff08
diff --git a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/SafetyStatusPreference.java b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/SafetyStatusPreference.java
index 3242ea8..c022daa 100644
--- a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/SafetyStatusPreference.java
+++ b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/SafetyStatusPreference.java
@@ -75,7 +75,7 @@
@Override
public void onBindViewHolder(PreferenceViewHolder holder) {
super.onBindViewHolder(holder);
- Log.d(TAG, String.format("onBindViewHolder called for status %s", mStatus));
+ Log.v(TAG, String.format("onBindViewHolder called for status %s", mStatus));
if (mStatus == null) {
return;
@@ -144,12 +144,14 @@
if (mIsTextChangeAnimationRunning) {
return;
}
+ Log.v(TAG, "Starting status text animation");
String titleText = mStatus.getTitle().toString();
String summaryText = mStatus.getSummary(getContext()).toString();
boolean titleEquals = titleView.getText().toString().equals(titleText);
boolean summaryEquals = summaryView.getText().toString().equals(summaryText);
Runnable onFinish =
() -> {
+ Log.v(TAG, "Finishing status text animation");
mIsTextChangeAnimationRunning = false;
runTextAnimationIfNeeded(titleView, summaryView);
};
@@ -339,7 +341,7 @@
void setData(StatusUiData statusUiData) {
mStatus = statusUiData;
- Log.d(TAG, String.format("setData called for status %s", mStatus));
+ Log.v(TAG, String.format("setData called for status %s", mStatus));
safeNotifyChanged();
}
@@ -357,7 +359,7 @@
new Handler(Looper.getMainLooper())
.post(
() -> {
- Log.d(
+ Log.v(
TAG,
String.format("Calling notifyChanged for status %s", mStatus));
notifyChanged();
diff --git a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/TextFadeAnimator.kt b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/TextFadeAnimator.kt
index 656c380..48acad4 100644
--- a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/TextFadeAnimator.kt
+++ b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/TextFadeAnimator.kt
@@ -21,6 +21,7 @@
import android.transition.TransitionListenerAdapter
import android.transition.TransitionManager
import android.transition.TransitionSet
+import android.util.Log
import android.view.View
import android.view.ViewGroup
import android.view.animation.LinearInterpolator
@@ -64,6 +65,9 @@
if (textChanges.isEmpty()) {
return
}
+
+ Log.v(TAG, "Starting text animation")
+
val firstView = textChanges[0].first
val parentViewGroup: ViewGroup = firstView.parent as ViewGroup
val fadeOutTransition =
@@ -71,17 +75,14 @@
.clone()
.addListener(
object : TransitionListenerAdapter() {
- override fun onTransitionStart(transition: Transition?) {
- super.onTransitionStart(transition)
- }
override fun onTransitionEnd(transition: Transition?) {
- super.onTransitionEnd(transition)
fadeTextIn(textChanges, parentViewGroup, onFinish)
}
}
)
parentViewGroup.post {
TransitionManager.beginDelayedTransition(parentViewGroup, fadeOutTransition)
+ Log.v(TAG, "Starting text fade-out transition")
for ((textView, _) in textChanges) {
textView.visibility = View.INVISIBLE
}
@@ -99,7 +100,7 @@
.addListener(
object : TransitionListenerAdapter() {
override fun onTransitionEnd(transition: Transition?) {
- super.onTransitionEnd(transition)
+ Log.v(TAG, String.format("Finishing text animation"))
onFinish?.run()
}
}
@@ -107,6 +108,7 @@
parent.post {
TransitionManager.beginDelayedTransition(parent, fadeInTransition)
+ Log.v(TAG, "Starting text fade-in transition")
for ((textView, text) in textChanges) {
textView.text = text
textView.visibility = View.VISIBLE
@@ -119,6 +121,7 @@
}
companion object {
+ private const val TAG = "TextFadeAnimator"
// Duration is for fade-out & fade-in individually, not combined
private val DEFAULT_TEXT_CHANGE_DURATION = Duration.ofMillis(167)
private val linearInterpolator = LinearInterpolator()
diff --git a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/view/MoreIssuesHeaderView.kt b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/view/MoreIssuesHeaderView.kt
index 00cfd14..b8142e1 100644
--- a/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/view/MoreIssuesHeaderView.kt
+++ b/PermissionController/src/com/android/permissioncontroller/safetycenter/ui/view/MoreIssuesHeaderView.kt
@@ -168,6 +168,7 @@
if (animateTextChange) {
counterView.text = previousText
+ Log.v(TAG, "Starting more issues card text animation")
moreIssuesCardAnimator.animateChangeText(counterView, newText)
} else {
counterView.text = newText
