[Voice Broadcast] Move the live broadcast computation in the room list behind the lab flag

[Florian14]

Type of change

• Feature
• Bugfix
• Technical
• Other :

Content

• The computation of the Live broadcast label in the room list is moved behind the voice broadcast lab flag. This is a temporary workaround before changing the current implementation.
• Improve existing use case by removing a useless blocking call on a Kotlin Flow.

Motivation and context

The computation of the Live broadcast label introduced performance issues in the rendering of the room list because it requires a lot of DB queries as this cannot be done at the sdk level by updating the RoomSummaryEntity.

Screenshots / GIFs

Tests

• Start a voice broadcast in a room
• Verify that the live broadcast label is displayed or not in the room list according to the lab flag status

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()

[bmarty]

LGTM, thanks!

[yostyle]

LGTM

[sonarcloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

75.0% Coverage
0.0% Duplication