Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Remove unused notification pipeline flag code" into tm-qpr-dev am: 241b46e657

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/20176971

Change-Id: I335761f448698aa61ff5d4e4092ee9ed72b50ab5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
TreeHugger Robot
2022-10-20 04:55:21 +00:00
committed by Automerger Merge Worker
parent 10a23b2b8b 241b46e657
commit d05f2ef342
2 changed files with 0 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/SystemUI/src/com/android/systemui/flags/Flags.java
View File

@@ -54,9 +54,6 @@ public class Flags {
public static final UnreleasedFlag NSSL_DEBUG_REMOVE_ANIMATION = public static final UnreleasedFlag NSSL_DEBUG_REMOVE_ANIMATION =
new UnreleasedFlag(106); new UnreleasedFlag(106);
public static final UnreleasedFlag NEW_PIPELINE_CRASH_ON_CALL_TO_OLD_PIPELINE =
new UnreleasedFlag(107);
public static final ResourceBooleanFlag NOTIFICATION_DRAG_TO_CONTENTS = public static final ResourceBooleanFlag NOTIFICATION_DRAG_TO_CONTENTS =
new ResourceBooleanFlag(108, R.bool.config_notificationToContents); new ResourceBooleanFlag(108, R.bool.config_notificationToContents);

16
packages/SystemUI/src/com/android/systemui/statusbar/notification/NotifPipelineFlags.kt
View File

@@ -17,30 +17,14 @@
package com.android.systemui.statusbar.notification package com.android.systemui.statusbar.notification
import android.content.Context import android.content.Context
import android.util.Log
import android.widget.Toast
import com.android.systemui.flags.FeatureFlags import com.android.systemui.flags.FeatureFlags
import com.android.systemui.flags.Flags import com.android.systemui.flags.Flags
import com.android.systemui.util.Compile
import javax.inject.Inject import javax.inject.Inject
class NotifPipelineFlags @Inject constructor( class NotifPipelineFlags @Inject constructor(
val context: Context, val context: Context,
val featureFlags: FeatureFlags val featureFlags: FeatureFlags
) { ) {
fun checkLegacyPipelineEnabled(): Boolean {
if (Compile.IS_DEBUG) {
Toast.makeText(context, "Old pipeline code running!", Toast.LENGTH_SHORT).show()
}
if (featureFlags.isEnabled(Flags.NEW_PIPELINE_CRASH_ON_CALL_TO_OLD_PIPELINE)) {
throw RuntimeException("Old pipeline code running with new pipeline enabled")
} else {
Log.d("NotifPipeline", "Old pipeline code running with new pipeline enabled",
Exception())
}
return false
}
fun isDevLoggingEnabled(): Boolean = fun isDevLoggingEnabled(): Boolean =
featureFlags.isEnabled(Flags.NOTIFICATION_PIPELINE_DEVELOPER_LOGGING) featureFlags.isEnabled(Flags.NOTIFICATION_PIPELINE_DEVELOPER_LOGGING)