VIDEO: Fix seeking to a keyframe in BINK videos

[antoniou79]

When seeking to a keyframe, that is not the first frame of the video, audio should not break or desync

The special code in BinkDecoder::BinkAudioTrack::seek() for fixing audio underrun should apply in the case of key frames as well

A test case for this issue can be constructed in MYST3, if the video at the start (10002), when Catherine is sitting on the bench with the baby and talks to the stranger, is set to start at frame 1 instead of 2.

If the fix is correct, it allows for some (future) fixes in the video playing code for Myst 3, that are currently breaking due to this audio issue.

[lephilousophe]

Without knowing anything about the codec, this seems completely sensible.
The first part of the function seeks to keyFrame and then, if the target frame is not a keyframe, we decode the progressive frames up to the target and skip the proper amount of audio samples.

[aquadran]

Without knowing anything about the codec, this seems completely sensible. The first part of the function seeks to keyFrame and then, if the target frame is not a keyframe, we decode the progressive frames up to the target and skip the proper amount of audio samples.

unfortunately I can't add anything more, but also sensible for me.