web/video.js
1
0
Fork 0
mirror of https://github.com/videojs/video.js.git synced 2024-12-14 11:23:30 +02:00
Code Releases Activity

fix: Patch a memory leak caused by un-removed track listener(s). (#3976)

Browse Source
This commit is contained in:
Pat O'Neill 2017-01-27 14:56:43 -05:00 committed by Gary Katsevman
parent 3585af0fe2
commit 4979ea78d5
1 changed files with 14 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/js/tech/tech.js
View File

@ -559,14 +559,12 @@ class Tech extends Component {
*/
emulateTextTracks() {
const tracks = this.textTracks();
const remoteTracks = this.remoteTextTracks();
const handleAddTrack = (e) => tracks.addTrack(e.track);
const handleRemoveTrack = (e) => tracks.removeTrack(e.track);
this.remoteTextTracks().on('addtrack', (e) => {
tracks.addTrack(e.track);
});
this.remoteTextTracks().on('removetrack', (e) => {
tracks.removeTrack(e.track);
});
remoteTracks.on('addtrack', handleAddTrack);
remoteTracks.on('removetrack', handleRemoveTrack);
// Initially, Tech.el_ is a child of a dummy-div wait until the Component system
// signals that the Tech is ready at which point Tech.el_ is part of the DOM
@ -574,6 +572,7 @@ class Tech extends Component {
this.on('ready', this.addWebVttScript_);
const updateDisplay = () => this.trigger('texttrackchange');
const textTracksChanges = () => {
updateDisplay();
@ -591,7 +590,15 @@ class Tech extends Component {
tracks.addEventListener('change', textTracksChanges);
this.on('dispose', function() {
remoteTracks.off('addtrack', handleAddTrack);
remoteTracks.off('removetrack', handleRemoveTrack);
tracks.removeEventListener('change', textTracksChanges);
for (let i = 0; i < tracks.length; i++) {
const track = tracks[i];
track.removeEventListener('cuechange', updateDisplay);
}
});
}