feat(mpris): Get position from player and provide it to MPRIS

Author: paulfariello

src/mpris_event_handler.rs

@@ -563,6 +563,7 @@ struct MprisPlayerService {
     shuffle: bool,
     playback_status: PlaybackStatus,
     volume: u16,
+    position_ms: Option<u32>,
     metadata: Metadata,
 }
 
@@ -905,7 +906,9 @@ impl MprisPlayerService {
     async fn position(&self) -> zbus::fdo::Result<TimeInUs> {
         debug!("org.mpris.MediaPlayer2.Player::Position");
         // todo!("fetch up-to-date position from player")
-        Ok(0)
+        self.position_ms
+            .map(|position_ms| position_ms as i64 * 1000)
+            .ok_or(zbus::fdo::Error::Failed(String::from("Got no position")))
     }
 
     // The minimum value which the `Rate` property can take. Clients should not attempt to set the
@@ -1104,6 +1107,7 @@ impl MprisEventHandler {
             shuffle: false,
             playback_status: PlaybackStatus::Stopped,
             volume: u16::MAX,
+            position_ms: None,
             metadata: Metadata::default(),
         };
 
@@ -1261,12 +1265,14 @@ impl MprisTask {
             }
             PlayerEvent::Playing {
                 track_id,
-                // position_ms,
+                position_ms,
                 ..
             } => {
-                // TODO: update position
                 let iface_ref = self.mpris_player_iface().await;
                 let mut iface = iface_ref.get_mut().await;
+
+                iface.position_ms = Some(position_ms);
+
                 let meta = &mut iface.metadata;
 
                 if meta.mpris.track_id.as_ref() != Some(&track_id) {
@@ -1283,12 +1289,14 @@ impl MprisTask {
             }
             PlayerEvent::Paused {
                 track_id,
-                // position_ms,
+                position_ms,
                 ..
             } => {
-                // TODO: update position
                 let iface_ref = self.mpris_player_iface().await;
                 let mut iface = iface_ref.get_mut().await;
+
+                iface.position_ms = Some(position_ms);
+
                 let meta = &mut iface.metadata;
 
                 if meta.mpris.track_id.as_ref() != Some(&track_id) {
@@ -1330,12 +1338,14 @@ impl MprisTask {
             }
             PlayerEvent::Seeked {
                 track_id,
-                // position_ms,
+                position_ms,
                 ..
             } => {
-                // TODO: Update position
                 let iface_ref = self.mpris_player_iface().await;
                 let mut iface = iface_ref.get_mut().await;
+
+                iface.position_ms = Some(position_ms);
+
                 let meta = &mut iface.metadata;
 
                 if meta.mpris.track_id.as_ref() != Some(&track_id) {
@@ -1347,11 +1357,14 @@ impl MprisTask {
             }
             PlayerEvent::PositionCorrection {
                 track_id,
-                // position_ms,
+                position_ms,
                 ..
             } => {
                 let iface_ref = self.mpris_player_iface().await;
                 let mut iface = iface_ref.get_mut().await;
+
+                iface.position_ms = Some(position_ms);
+
                 let meta = &mut iface.metadata;
 
                 if meta.mpris.track_id.as_ref() != Some(&track_id) {
@@ -1361,9 +1374,16 @@ impl MprisTask {
                     iface.metadata_changed(iface_ref.signal_context()).await?;
                 }
             }
-            PlayerEvent::PositionChanged { track_id, .. } => {
+            PlayerEvent::PositionChanged {
+                track_id,
+                position_ms,
+                ..
+            } => {
                 let iface_ref = self.mpris_player_iface().await;
                 let mut iface = iface_ref.get_mut().await;
+
+                iface.position_ms = Some(position_ms);
+
                 let meta = &mut iface.metadata;
 
                 if meta.mpris.track_id.as_ref() != Some(&track_id) {