From 3b7cb6cbd0548ad82483fd9831221c09dcaebc7d Mon Sep 17 00:00:00 2001 From: Caitlin Shkuratov Date: Mon, 14 Aug 2023 16:26:56 +0000 Subject: [PATCH] [SB][Mobile] Remove logs for activity in/out visibility. After reading various mobile bug reports, it seems like the activity logs are just noise that we don't need. Bug: 238425913 Test: verify activity logs aren't in table dump Change-Id: I79151501eedb36a647435259764d1b727b6d6b9d --- .../ui/viewmodel/MobileIconViewModel.kt | 21 ------------------- 1 file changed, 21 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/pipeline/mobile/ui/viewmodel/MobileIconViewModel.kt b/packages/SystemUI/src/com/android/systemui/statusbar/pipeline/mobile/ui/viewmodel/MobileIconViewModel.kt index fe2481595ff40..275cfc58d5818 100644 --- a/packages/SystemUI/src/com/android/systemui/statusbar/pipeline/mobile/ui/viewmodel/MobileIconViewModel.kt +++ b/packages/SystemUI/src/com/android/systemui/statusbar/pipeline/mobile/ui/viewmodel/MobileIconViewModel.kt @@ -221,36 +221,15 @@ constructor( override val activityInVisible: Flow = activity .map { it?.hasActivityIn ?: false } - .distinctUntilChanged() - .logDiffsForTable( - iconInteractor.tableLogBuffer, - columnPrefix = "", - columnName = "activityInVisible", - initialValue = false, - ) .stateIn(scope, SharingStarted.WhileSubscribed(), false) override val activityOutVisible: Flow = activity .map { it?.hasActivityOut ?: false } - .distinctUntilChanged() - .logDiffsForTable( - iconInteractor.tableLogBuffer, - columnPrefix = "", - columnName = "activityOutVisible", - initialValue = false, - ) .stateIn(scope, SharingStarted.WhileSubscribed(), false) override val activityContainerVisible: Flow = activity .map { it != null && (it.hasActivityIn || it.hasActivityOut) } - .distinctUntilChanged() - .logDiffsForTable( - iconInteractor.tableLogBuffer, - columnPrefix = "", - columnName = "activityContainerVisible", - initialValue = false, - ) .stateIn(scope, SharingStarted.WhileSubscribed(), false) }