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

Commit 68d87728 authored by burakov's avatar burakov Committed by Danny Burakov
Browse files

[Dual Shade] Remove the redundant coercion of the shade expand fraction. 

The `expandFraction` should be in [0..1] per its API contract, so the
caller shouldn't have to coerce the value.

Bug: 400530515
Test: Unit tests still pass.
Flag: com.android.systemui.scene_container
Change-Id: I3ed890ee4f16ca7fda2cc65d1cd0c542445bbade
parent 0fc21121

packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/ui/viewbinder/NotificationScrollViewBinder.kt

+1 −5
Original line number Diff line number Diff line
@@ -90,11 +90,7 @@ constructor( 


            launch { viewModel.maxAlpha.collectTraced { view.setMaxAlpha(it) } }

            launch { viewModel.shadeScrollState.collect { view.setScrollState(it) } }

            launch {

                viewModel.expandFraction.collectTraced {

                    view.setExpandFraction(it.coerceIn(0f, 1f))

                }

            }

            launch { viewModel.expandFraction.collectTraced { view.setExpandFraction(it) } }

            launch { viewModel.qsExpandFraction.collectTraced { view.setQsExpandFraction(it) } }

            launch { viewModel.blurRadius(maxBlurRadius).collect(view::setBlurRadius) }

            launch {