From 438d28c0ac22d8e70f5232deac75e4651875b23c Mon Sep 17 00:00:00 2001 From: Hawkwood Glazier Date: Wed, 22 Jun 2022 18:48:22 +0000 Subject: [PATCH] Add wtf log for UMO being cut Bug: 232528163 Test: Manual Change-Id: Ide3e8d63377b9b224f010420296e36b6f70be298 --- .../android/systemui/media/MediaHierarchyManager.kt | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/packages/SystemUI/src/com/android/systemui/media/MediaHierarchyManager.kt b/packages/SystemUI/src/com/android/systemui/media/MediaHierarchyManager.kt index ed5c1933af251..2f732de50ea1c 100644 --- a/packages/SystemUI/src/com/android/systemui/media/MediaHierarchyManager.kt +++ b/packages/SystemUI/src/com/android/systemui/media/MediaHierarchyManager.kt @@ -23,6 +23,7 @@ import android.annotation.IntDef import android.content.Context import android.content.res.Configuration import android.graphics.Rect +import android.util.Log import android.util.MathUtils import android.view.View import android.view.ViewGroup @@ -48,6 +49,8 @@ import com.android.systemui.util.animation.UniqueObjectHostView import com.android.systemui.util.traceSection import javax.inject.Inject +private val TAG: String = MediaHierarchyManager::class.java.simpleName + /** * Similarly to isShown but also excludes views that have 0 alpha */ @@ -964,6 +967,14 @@ class MediaHierarchyManager @Inject constructor( top, left + currentBounds.width(), top + currentBounds.height()) + + if (mediaFrame.childCount > 0) { + val child = mediaFrame.getChildAt(0) + if (mediaFrame.height < child.height) { + Log.wtf(TAG, "mediaFrame height is too small for child: " + + "${mediaFrame.height} vs ${child.height}") + } + } } if (isCrossFadeAnimatorRunning) { // When cross-fading with an animation, we only notify the media carousel of the