[MattiasBuelens]

Summary

While scrubbing the SeekBar, SeekBarState.onValueChange used to set player.currentTime on every position change: once per drag frame on mobile, and once per key repeat on TV. That loaded media for positions that were only scrubbed past.

Now the player only seeks in two cases:

• the scrub position stays the same for 250 ms;
• the scrub finishes (onValueChangeFinished). This seek is skipped if the debounced seek already went to the same position.

seekTo (a single TV Left/Right press, or a tap) still seeks immediately.

Without the intermediate seeks, PlaybackTimeState wouldn't know where the user is scrubbing to. To solve that, SeekBarState now extends PlaybackTimeState:

open class PlaybackTimeState internal constructor(player: Player?) {
    open val currentTime: Double get() = playerTime
}
class SeekBarState internal constructor(player: Player?) : PlaybackTimeState(player) {
    val scrubTime: Double?                             // absolute scrub position, or null
    override val currentTime get() = scrubTime ?: playerTime
}

• Explicit wiring: CurrentTimeDisplay(state = seekBarState) follows the thumb.
• Default overloads: UIControllerCore remembers one SeekBarState and provides it as LocalUIControllerSeekBarState (@InternalUiApi).
  • The default SeekBar() overloads use it through rememberUIControllerSeekBarState(player).
  • rememberPlaybackTimeState(player) returns it when it belongs to the same player.
  • So SeekBar() and CurrentTimeDisplay() inside a UIController share one state.

Stacked on #148.

Link to Devin session: https://dolby.devinenterprise.com/sessions/63210218adb6456ab0406d4eb65d28e9
Open in Devin Desktop: https://dolby.devinenterprise.com/desktop/session/63210218adb6456ab0406d4eb65d28e9?variant=devin
Requested by: @MattiasBuelens

[devin-ai-integration]

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

• Disable automatic comment, CI, and merge conflict monitoring