change(ffi): remove the room's timeline method in favor of timeline_with_configuration

Author: stefanceriu

bindings/matrix-sdk-ffi/src/room.rs

@@ -13,7 +13,7 @@ use matrix_sdk::{
     RoomHero as SdkRoomHero, RoomMemberships, RoomState,
 };
 use matrix_sdk_ui::{
-    timeline::{default_event_filter, RoomExt, TimelineBuilder},
+    timeline::{default_event_filter, TimelineBuilder},
     unable_to_decrypt_hook::UtdHookManager,
 };
 use mime::Mime;
@@ -31,7 +31,6 @@ use ruma::{
     },
     EventId, Int, OwnedDeviceId, OwnedUserId, RoomAliasId, UserId,
 };
-use tokio::sync::RwLock;
 use tracing::{error, warn};
 
 use crate::{
@@ -73,18 +72,15 @@ impl From<RoomState> for Membership {
     }
 }
 
-pub(crate) type TimelineLock = Arc<RwLock<Option<Arc<Timeline>>>>;
-
 #[derive(uniffi::Object)]
 pub struct Room {
     pub(super) inner: SdkRoom,
     utd_hook_manager: Option<Arc<UtdHookManager>>,
-    timeline: TimelineLock,
 }
 
 impl Room {
     pub(crate) fn new(inner: SdkRoom, utd_hook_manager: Option<Arc<UtdHookManager>>) -> Self {
-        Room { inner, timeline: Default::default(), utd_hook_manager }
+        Room { inner, utd_hook_manager }
     }
 }
 
@@ -173,17 +169,6 @@ impl Room {
         Ok(())
     }
 
-    pub async fn timeline(&self) -> Result<Arc<Timeline>, ClientError> {
-        let mut write_guard = self.timeline.write().await;
-        if let Some(timeline) = &*write_guard {
-            Ok(timeline.clone())
-        } else {
-            let timeline = Timeline::new(self.inner.timeline().await?);
-            *write_guard = Some(timeline.clone());
-            Ok(timeline)
-        }
-    }
-
     /// Build a new timeline instance with the given configuration.
     pub async fn timeline_with_configuration(
         &self,