Fix memory leak (#7097)

chartjs · Feb 13, 2020 · 72c3c6d · 72c3c6d
1 parent 25a9969
commit 72c3c6d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/src/core/core.animator.js b/src/core/core.animator.js
@@ -210,6 +210,14 @@ class Animator {
 		anims.items = [];
 		this._notify(chart, anims, Date.now(), 'complete');
 	}
+
+	/**
+	 * Remove chart from Animator
+	 * @param {Chart} chart
+	 */
+	remove(chart) {
+		return this._charts.delete(chart);
+	}
 }
 
 const instance = new Animator();

diff --git a/src/core/core.controller.js b/src/core/core.controller.js
@@ -893,6 +893,7 @@ class Chart {
 		let i, ilen;
 
 		me.stop();
+		Animator.remove(me);
 
 		// dataset controllers need to cleanup associated data
 		for (i = 0, ilen = me.data.datasets.length; i < ilen; ++i) {