wangzuo / react-videojs

react videojs wrapper

Home Page:http://wangzuo.github.io/react-videojs/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Does not call player.dispose() on unMount - potential memory leak

Mike-Bell opened this issue · comments

On componentWillUnmount, we should call this.player && this.player.dispose();. Failing to do so will lead to memory leaks in the scenario that the component is dynamically added and removed on your page (found this out the hard way).

See http://docs.videojs.com/Player.html#dispose

seems safe to say this will not be fixed