https://bugs.kde.org/show_bug.cgi?id=469617

            Bug ID: 469617
           Summary: MPRIS receivers/controllers are ignoring playback rate
                    to extrapolate current play position
    Classification: Applications
           Product: kdeconnect
           Version: unspecified
          Platform: Other
                OS: All
            Status: REPORTED
          Severity: normal
          Priority: NOR
         Component: common
          Assignee: albertv...@gmail.com
          Reporter: b...@mogwai.be
                CC: andrew.g.r.hol...@gmail.com
  Target Milestone: ---

SUMMARY
The MPRIS receiver/send implementation is ignoring the playback rate settings. 
This is at least the case in the linux and android implementations.
As a result, if a user is using a media application with a playback rate
setting other than 1, the kdeconnect media controller applet is showing an
incorrect position since it simply adds 1 second to the play position every
second.  Hence, the playback position starts to run out-of-sync with the real
media player.
Note that the MPRIS2 DBus specification says to only to update the Position
property if playback does not proceed at the currently set playback rate value.
 Hence, it is implied that the extrapolated playback position should take into
account the playback rate.
I checked the source code of both the linux and android app, and there are
simply no handlers for the MPRIS2 Rate property.

STEPS TO REPRODUCE
1. Start up a media application with variable playback speed control (on e.g.
linux).  Start playing back something and change the playback rate != 1.
2. Open the kdeconnect-android media controller and keep an eye on the play
position wrt the play position on the originating application on linux.

OBSERVED RESULT
Watch the playback position drift out-of-syn.

EXPECTED RESULT
The extrapolated playback position should take into account the Rate property
value.

ADDITIONAL INFORMATION
The MRPIS2 Rate property should be implemented and the extrapolated playback
position should take this value into account.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to