Correctly update Chronometer in PlayerNotificationManager for playback speeds != 1.0

[Cosminnv]

[REQUIRED] Searched documentation and issues
Searched in old issues/ google/ documentation but found nothing.

[REQUIRED] Question

I have a video playing app where you can set the speed of the video. I also have a PlayerManagerNotification to allow the user to listen while the app is in background.

Problem:
The problem is that when i change the speed of the video via PlaybackParameters the video itself works fine, but the notification current time isn't in sync with the current time of the video.

Example:
For example, if i watch a video at x2 speed for 10 real life seconds, i watch 20 seconds of the video, but the notification watch time is still at 10 seconds.

Here's my setup:
I have a single exoplayer instance binded to an AudioPlayerService. Everything is in sync and works as expected, apart from the current time when the speed is changed.

Here's how i set the Playback Speed:
Constants.player.setPlaybackParameters(new PlaybackParameters(2.0f));

I don't know if i explained that properly, if you don't understand i can register a video.

(Also Happy New Year 😋 )

[tonihei]

Thanks for reporting. This is in some way working as intended because Android's notification uses a Chronometer to show the current time. As you can see in the docs, this text widget is always counting in real time irrespective of the playback speed.

To avoid this problem, we should probably disable the chronometer for playbacks with speed != 1.0 in all cases. A more advanced fix is to update the time manually every second or so. I'll mark this as an enhancement to implement this feature.

[ojw28]

I think we should just disable it. I actually thought we did this already, but apparently not! The more advanced fix may still produce weird effects, depending on the UI that's on the other side.

[Cosminnv]

Thanks for reporting. This is in some way working as intended because Android's notification uses a Chronometer to show the current time. As you can see in the docs, this text widget is always counting in real time irrespective of the playback speed.

To avoid this problem, we should probably disable the chronometer for playbacks with speed != 1.0 in all cases. A more advanced fix is to update the time manually every second or so. I'll mark this as an enhancement to implement this feature.

Oh so this isn't my fault then, i was going mad over this. Thanks for the explanation !