Persist the playback state of voice messages across different screens

[Florian14]

Type of change

• Feature
• Bugfix
• Technical
• Other :

Content

When pausing a voice message playback, the playback position is now persisted when closing the room timeline.
This also affects voice broadcast playbacks.

Motivation and context

Voice Broadcast feature requires to be able to listen to a voice broadcast when navigating across the app. For this need, we decided to not reset the playback position for voice messages when closing the room timeline.

The positions will be reset only after killing the application.

Screenshots / GIFs

Tests

• Have a room timeline with several voice broadcasts and voice messages
• Play/pause some of them or seek to a given position
• Navigate into the app
• Go back to the previous room
• Verify that the playback positions are not reset
• Verify that the position is correct when resuming the playback

Tested devices

• Physical
• Emulator
• OS version(s):

Checklist

• Changes has been tested on an Android device or Android emulator with API 21
• UI change has been tested on both light and dark themes
• Accessibility has been taken into account. See https://github.com/vector-im/element-android/blob/develop/CONTRIBUTING.md#accessibility
• Pull request is based on the develop branch
• Pull request includes a new file under ./changelog.d. See https://github.com/vector-im/element-android/blob/develop/CONTRIBUTING.md#changelog
• Pull request includes screenshots or videos if containing UI changes
• Pull request includes a sign off
• You've made a self review of your PR
• If you have modified the screen flow, or added new screens to the application, you have updated the test UiAllScreensSanityTest.allScreensTest()

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

82.0% Coverage
0.0% Duplication

[giomfo]

Approved
I just noticed the playback positions will survive to a clear cache. Personally I don't know if this is good or not. We may wait before deciding to clear or not this information on clear cache.
Note: these information are already reset on a new app launch, which seems enough