tree d40fe6c871d74db19525a863319b4cf09c838aeb
parent dd1ce567b54196ad403eefe46524687c7cda546c
author Alexander Roederer <aroederer@google.com> 1727977130 +0000
committer Alexander Roederer <aroederer@google.com> 1728067709 +0000

Fix race on setCanceledAfterLifetimeExtension

We setCancaledAfterLifetimeExtension to avoid multiple cancelation calls
from causing multiple updates being sent to System UI, but were setting
the value on a callback after the post to System UI had occurred.
Unfortunately apps can cancel more quickly than that; these cancelations
will enqueue multiple posts before the callback from systemUI occurs,
and they'll all be sent.

This leads to the same message being posted repeatedly.

So, instead, we move the set to immediately after we've checked
that the record doesn't have the value set, and have committed to the path
where an update will be sent to system UI.

Bug: 371105236
Test: atest NotificationManagerService, flash+test
Flag: android.app.lifetime_extension_refactor
Change-Id: I82d0dfc9940f02108b61f4d8574c9135505efa58