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

Commit 338ade29 authored by Ioana Alexandru's avatar Ioana Alexandru Committed by Automerger Merge Worker
Browse files

Merge "Don't attach pipeline to group expansion mgr." into udc-dev am: d77e6968 am: 56f4f108 

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



Change-Id: I23642cec741850834fc8e7da3daec9608cfd75a8
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents ce64d242 56f4f108

packages/SystemUI/src/com/android/systemui/statusbar/notification/collection/coordinator/StackCoordinator.kt

+4 −1
Original line number Diff line number Diff line
@@ -39,7 +39,10 @@ class StackCoordinator @Inject internal constructor( 


    override fun attach(pipeline: NotifPipeline) {

        pipeline.addOnAfterRenderListListener(::onAfterRenderList)

        groupExpansionManagerImpl.attach(pipeline)

        // TODO(b/282865576): This has an issue where it makes changes to some groups without

        // notifying listeners. To be fixed in QPR, but for now let's comment it out to avoid the

        // group expansion bug.

        // groupExpansionManagerImpl.attach(pipeline)

    }



    fun onAfterRenderList(entries: List<ListEntry>, controller: NotifStackController) =