diff options
| author | 2025-02-03 13:47:42 +0000 | |
|---|---|---|
| committer | 2025-02-03 17:52:09 +0000 | |
| commit | 165c5c88601791d852fdbc8abf1e2e9625fa0c18 (patch) | |
| tree | e9669ca9955d59652c8bc530436a289dde7d97db /libs/androidfw/Util.cpp | |
| parent | 7acbff030c88c60c9a99a114a76d9c2d53478e1d (diff) | |
Fix race condition in NotificationStatsLogger to avoid dupe logs
Ensure that notification visibility state changes are processed
sequentially to avoid duplicate logging.
When the last HUN is animating away over launcher, and the shade becomes
non-interactible again, the NotificationStatsLogger receives two updates
over a short time. An onNotificationLocationsChanged call saying that
the HUN is no longer visible, and an onLockscreenOrShadeNotInteractive
call indicating that the HUN is not visible anymore.
The second update didn't wait for the previous job to complete, and it
was getting started before we clean up the last reported array. This was
leading to duplicate logs with the same data.
Fixes: 376667852
Test: Remove a HUN, and observe the logs
- receive a HUN while the shade is closed
- wait until it animates away
- check that there is no duplicate IStatusBarService log
Flag: com.android.systemui.notifications_live_data_store_refactor
Change-Id: Ia9f5b40c77b9120192d5c02964967b42d300b125
Diffstat (limited to 'libs/androidfw/Util.cpp')
0 files changed, 0 insertions, 0 deletions