Skip to content

Commit

Permalink
fix: Failed to set 'currentTime' property on 'HTMLMediaElement' on a …
Browse files Browse the repository at this point in the history
…Hisense TV (#4962)

Fixes #4888
  • Loading branch information
avelad authored and joeyparrish committed Feb 9, 2023
1 parent 15afb4b commit bda4e50
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions lib/media/video_wrapper.js
Original file line number Diff line number Diff line change
Expand Up @@ -135,15 +135,15 @@ shaka.media.VideoWrapper = class {
// calling |Player.load|, meaning that |currentTime| is more meaningful than
// |startTime|.
//
// Seeking to the current time is a work around for Issue 1298. If we don't
// do this, the video may get stuck and not play.
// Seeking to the current time is a work around for Issue 1298 and 4888.
// If we don't do this, the video may get stuck and not play.
//
// TODO: Need further investigation why it happens. Before and after
// setting the current time, video.readyState is 1, video.paused is true,
// and video.buffered's TimeRanges length is 0.
// See: https://github.com/shaka-project/shaka-player/issues/1298
this.mover_.moveTo(
this.video_.currentTime == 0 ?
(!this.video_.currentTime || this.video_.currentTime == 0) ?
startTime :
this.video_.currentTime);
}
Expand Down

0 comments on commit bda4e50

Please sign in to comment.