tree 91e241e777404f07f60489105796962bc6e0ec03
parent 0cba866c8519e16c67e7f6954aba3b808cebda32
author Zim <zezeozue@google.com> 1646327860 +0000
committer Zim <zezeozue@google.com> 1646676688 +0000

Fix cloud media changed event notification delay

We throttled cloud media change event notifications by 1s such that we
sync at *most* once a second regardless of the number of
notifications.

There was a bug with the earlier implementation which created a lambda
each time as the Handler callback. This meant that we removed a 'new
callback' from the handler postDelayed method and the 1s throttle
wasn't actually respected.

Now we cache the sync delay handler and re-use that each time. We also
extended the default sync delay to 5s since 1s was short enough to
have frequent syncs while syncing a lot of db items. In the future, we
can add an exponential backoff and start with no delay.

Test: Manually verified that we re-use the callback on each
notification
Bug: 218791501

Change-Id: I07d41e9240c8198aa263638c800d10c13b42bfa6