Fix JavascriptSubtitlesOctopus dispose

[dmitrylyzo]

JavascriptSubtitlesOctopus has the dispose call in the error handler.
This makes it hard to control the disposing flow of elements.

Changes

• 2 hacks to clear resources after JSO error occurs.
  It would be better to remove that dispose call. But we need to discuss this with the upstream.
• Dispose JSO first (before destroying the player).

Issues
JSO is trying to remove canvas from the video element, but we already removed the latter from the DOM:
https://github.com/jellyfin/JavascriptSubtitlesOctopus/blob/f4625ac313b318bd5d2e0ae18679ff516370bae6/src/subtitles-octopus.js#L863

Uncaught DOMException: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.
    at SubtitlesOctopus.self.dispose (http://.../web/JavascriptSubtitlesOctopus_dist_js_subtitles-octopus_js.33eb72ced6222e91e928.chunk.js:1616:29)
    at Worker.SubtitlesOctopus.self.workerError (http://.../web/JavascriptSubtitlesOctopus_dist_js_subtitles-octopus_js.33eb72ced6222e91e928.chunk.js:1113:12)

Steps To Reproduce

1. Copy SRT subtitles as ASS (change extension).
2. Refresh metadata.
3. Select bad ASS subtitles.
4. Play the video.
5. Error occurs 3 times.

There is an error with the error message history, but I will fix this in the next PR (just to make things atomic).

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.3% Duplication