316c6f2621efa6d90b940aaafd5216db59126425 - platform_frameworks_base

commit	316c6f2621efa6d90b940aaafd5216db59126425	[log] [tgz]
author	Steve Elliott <steell@google.com>	Mon Jun 08 21:16:01 2020 -0400
committer	Steve Elliott <steell@google.com>	Mon Jun 08 21:22:46 2020 -0400
tree	d29a22d6c77583fb66597e6a740bce3cd98e6eb2
parent	06174db579ff1e30825568b894e6172bc4b044fa [diff]

Fix occasional bad placement of Conversation header

This CL refactors the header placement logic to be "simpler", in that
we're abstracting a lot of the state manipulation behind a new inner
class, and are consolidating manipulation of that state to fewer
places in the overall algorithm.

Additional tests and comments have been added to hopefully further
improve robustness and clarity, respectively.

Fixes: 158475746
Test: atest, manual
Change-Id: I92a62cd5e46bfd1aff897b4189e53bc6768623d3

packages/SystemUI/src/com/android/systemui/log/dagger/LogModule.java[diff]
packages/SystemUI/src/com/android/systemui/statusbar/notification/stack/NotificationSectionsManager.kt[diff]
packages/SystemUI/src/com/android/systemui/util/ConvenienceExtensions.kt[diff]
packages/SystemUI/src/com/android/systemui/util/SparseArrayUtils.kt[diff]
packages/SystemUI/tests/src/com/android/systemui/statusbar/notification/stack/NotificationSectionsManagerTest.java[diff]

5 files changed