Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 5d52946f authored by Jeff DeCew's avatar Jeff DeCew
Browse files

New Pipeline: isNewPipelineEnabled() is always true. 

This just deletes Flags.NEW_NOTIFICATION_PIPELINE_RENDERING and returns true where it was used.

Merged-In: If2e36064a7b8dfc1f27ec046818ee68696a7e387
Change-Id: If2e36064a7b8dfc1f27ec046818ee68696a7e387
Test: attempt to disable flag; new pipeline stays on.
Bug: 200269355
parent f81f314c

packages/SystemUI/src/com/android/systemui/flags/Flags.java

+0 −3
Original line number Diff line number Diff line
@@ -45,9 +45,6 @@ public class Flags { 


    /***************************************/

    // 100 - notification

    public static final BooleanFlag NEW_NOTIFICATION_PIPELINE_RENDERING =

            new BooleanFlag(101, true);



    public static final BooleanFlag NOTIFICATION_PIPELINE_DEVELOPER_LOGGING =

            new BooleanFlag(103, false);

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

+1 −2
Original line number Diff line number Diff line
@@ -48,8 +48,7 @@ class NotifPipelineFlags @Inject constructor( 
    fun assertLegacyPipelineEnabled(): Unit =

        check(!isNewPipelineEnabled()) { "Old pipeline code running w/ new pipeline enabled" }



    fun isNewPipelineEnabled(): Boolean =

        featureFlags.isEnabled(Flags.NEW_NOTIFICATION_PIPELINE_RENDERING)

    fun isNewPipelineEnabled(): Boolean = true



    fun isDevLoggingEnabled(): Boolean =

        featureFlags.isEnabled(Flags.NOTIFICATION_PIPELINE_DEVELOPER_LOGGING)