1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin-web synced 2025-03-30 19:56:21 +00:00

Merge pull request #5718 from dmitrylyzo/videoplayer-cleanup

This commit is contained in:
Bill Thornton 2024-06-23 11:44:52 -04:00 committed by GitHub
commit 18061ce247
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -209,10 +209,6 @@ export class HtmlVideoPlayer {
* @type {number | null} * @type {number | null}
*/ */
#audioTrackIndexToSetOnPlaying; #audioTrackIndexToSetOnPlaying;
/**
* @type {null | undefined}
*/
#currentClock;
/** /**
* @type {any | null | undefined} * @type {any | null | undefined}
*/ */
@ -862,6 +858,8 @@ export class HtmlVideoPlayer {
videoElement.parentNode.removeChild(videoElement); videoElement.parentNode.removeChild(videoElement);
} }
this._currentAspectRatio = null;
const dlg = this.#videoDialog; const dlg = this.#videoDialog;
if (dlg) { if (dlg) {
this.#videoDialog = null; this.#videoDialog = null;
@ -1169,9 +1167,6 @@ export class HtmlVideoPlayer {
this.destroyNativeTracks(videoElement, targetTrackIndex); this.destroyNativeTracks(videoElement, targetTrackIndex);
this.destroyStoredTrackInfo(targetTrackIndex); this.destroyStoredTrackInfo(targetTrackIndex);
this.#currentClock = null;
this._currentAspectRatio = null;
const octopus = this.#currentAssRenderer; const octopus = this.#currentAssRenderer;
if (octopus) { if (octopus) {
octopus.dispose(); octopus.dispose();
@ -1499,16 +1494,6 @@ export class HtmlVideoPlayer {
* @private * @private
*/ */
updateSubtitleText(timeMs) { updateSubtitleText(timeMs) {
const clock = this.#currentClock;
if (clock) {
try {
clock.seek(timeMs / 1000);
} catch (err) {
console.error(`error in libjass: ${err}`);
}
return;
}
const allTrackEvents = [this.#currentTrackEvents, this.#currentSecondaryTrackEvents]; const allTrackEvents = [this.#currentTrackEvents, this.#currentSecondaryTrackEvents];
const subtitleTextElements = [this.#videoSubtitlesElem, this.#videoSecondarySubtitlesElem]; const subtitleTextElements = [this.#videoSubtitlesElem, this.#videoSecondarySubtitlesElem];