Sulkta-OSS/element-x-ada
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make sure any JoinedRustRoom is destroyed after use (#4678)

Browse source
This commit is contained in:
Jorge Martin Espinosa 2025-05-05 16:09:07 +02:00 • committed by GitHub
parent 2d189ae775
commit 1904c98c9a
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
5 changed files with 11 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
features/call/impl/src/main/kotlin/io/element/android/features/call/impl/ui/CallScreenPresenter.kt
View file

@ -241,7 +241,7 @@ class CallScreenPresenter @AssistedInject constructor(
private suspend fun MatrixClient.notifyCallStartIfNeeded(roomId: RoomId) {
if (!notifiedCallStart) {
getJoinedRoom(roomId)?.sendCallNotificationIfNeeded()
getJoinedRoom(roomId)?.use { it.sendCallNotificationIfNeeded() }
?.onSuccess { notifiedCallStart = true }
}
}

5
features/call/impl/src/main/kotlin/io/element/android/features/call/impl/utils/DefaultCallWidgetProvider.kt
View file

@ -47,8 +47,11 @@ class DefaultCallWidgetProvider @Inject constructor(
theme = theme,
).getOrThrow()
val driver = room.getWidgetDriver(widgetSettings).getOrThrow()
room.destroy()
CallWidgetProvider.GetWidgetResult(
driver = room.getWidgetDriver(widgetSettings).getOrThrow(),
driver = driver,
url = callUrl,
)
}

1
features/share/impl/src/main/kotlin/io/element/android/features/share/impl/SharePresenter.kt
View file

@ -86,6 +86,7 @@ class SharePresenter @AssistedInject constructor(
).isSuccess
}
.all { it }
.also { room.destroy() }
}
.all { it }
}

4
libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/room/JoinedRustRoom.kt
View file

@ -639,9 +639,11 @@ class JoinedRustRoom(
}
}
override fun close() = destroy()
override fun destroy() {
baseRoom.destroy()
liveInnerTimeline.close()
liveInnerTimeline.destroy()
roomCoroutineScope.cancel()
}

2
libraries/matrix/impl/src/main/kotlin/io/element/android/libraries/matrix/impl/room/RustBaseRoom.kt
View file

@ -94,6 +94,8 @@ class RustBaseRoom(
}
}
override fun close() = destroy()
override fun destroy() {
innerRoom.destroy()
}