From a98b58fe1ccbae1fc214c1b5dc5cbdcd538b6444 Mon Sep 17 00:00:00 2001 From: Jeff DeCew Date: Fri, 20 May 2022 13:43:49 +0000 Subject: [PATCH] Allow debug logging of NotificationStackSizeCalculator Bug: 232079794 Test: adb setprop log.tag.NotifStackSizeCalc VERBOSE ; adb shell am crash com.android.systemui ; adb logcat -s NotifStackSizeCalc Change-Id: Ibdea705b99eec40e3d8075483bddcc02db55209f --- .../notification/stack/NotificationStackSizeCalculator.kt | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackSizeCalculator.kt b/packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackSizeCalculator.kt index 9ea36d540f4d7..d05c3385e982e 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackSizeCalculator.kt +++ b/packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationStackSizeCalculator.kt @@ -28,14 +28,15 @@ import com.android.systemui.statusbar.StatusBarState.KEYGUARD import com.android.systemui.statusbar.SysuiStatusBarStateController import com.android.systemui.statusbar.notification.row.ExpandableNotificationRow import com.android.systemui.statusbar.notification.row.ExpandableView +import com.android.systemui.util.Compile import com.android.systemui.util.children import javax.inject.Inject import kotlin.math.max import kotlin.math.min import kotlin.properties.Delegates.notNull -private const val TAG = "NotificationStackSizeCalculator" -private const val DEBUG = false +private const val TAG = "NotifStackSizeCalc" +private val DEBUG = Compile.IS_DEBUG && Log.isLoggable(TAG, Log.DEBUG) /** Calculates number of notifications to display and the height of the notification stack. */ @SysUISingleton @@ -229,7 +230,7 @@ constructor( return true } - private fun log(s: () -> String) { + private inline fun log(s: () -> String) { if (DEBUG) { Log.d(TAG, s()) }