After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 703303 - Music doesn't properly tell MPRIS clients that the song is replayed when in repeat song mode
Music doesn't properly tell MPRIS clients that the song is replayed when in r...
Status: RESOLVED FIXED
Product: gnome-music
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: 3.10
Assigned To: Arnel Borja
gnome-music-maint
Depends on:
Blocks:
 
 
Reported: 2013-06-29 08:55 UTC by Arnel Borja
Modified: 2013-09-16 08:41 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
mpris: Seek to start when a song is loaded and in repeat song mode (1.13 KB, patch)
2013-09-15 05:31 UTC, Arnel Borja
committed Details | Review

Description Arnel Borja 2013-06-29 08:55:06 UTC
When setting the repeat mode to song, MPRIS clients adds up the position instead of resetting to 0 when the song reaches the end and replays.

To reproduce:
1. Play a song.
2. Set repeat mode to "repeat song".
3. Move position to end.

Result:   The current position is greater than duration in MPRIS clients.
Expected: MPRIS clients should show that the new position is start of song.

MediaPlayer2.Player interface of MPRIS says that Seeked signal "does not need to be emitted when playback starts or when the track changes, unless the track is starting at an unexpected position" so emitting it during start of playback is not the fix for this bug.

So to fix it, we could check how Rhythmbox handles this, by checking the dbus messages/signals produced using dbus-monitor when the song ends. That song should be inside a playlist with it the only item inside. The repeat mode of Rhythmbox should be enabled. We have to do this because Rhythmbox doesn't have a repeat song mode.
Comment 1 Arnel Borja 2013-09-15 05:31:31 UTC
Created attachment 254961 [details] [review]
mpris: Seek to start when a song is loaded and in repeat song mode

This makes sure that the current position will be reset when replaying
the song. This seems like it doesn't follow the MPRIS specifications (no
seeking when the media changes), but other media players do this too, so
do it anyway.
Comment 2 Vadim Rutkovsky 2013-09-16 07:50:30 UTC
Review of attachment 254961 [details] [review]:

Tested, please commit
Comment 3 Arnel Borja 2013-09-16 08:41:36 UTC
Comment on attachment 254961 [details] [review]
mpris: Seek to start when a song is loaded and in repeat song mode

Committed as 0da31ab5.