Merge pull request #1635 from ckeditor/t/ckeditor5/1341

Fix: There should be no memory leaks when the editor is created and destroyed (see ckeditor/ckeditor5#1341).

Author: oleq

src/controller/datacontroller.js

@@ -351,7 +351,9 @@ export default class DataController {
 	/**
 	 * Removes all event listeners set by the DataController.
 	 */
-	destroy() {}
+	destroy() {
+		this.stopListening();
+	}
 
 	/**
 	 * Checks if all provided root names are existing editor roots.

src/view/document.js

@@ -121,6 +121,14 @@ export default class Document {
 		this._postFixers.add( postFixer );
 	}
 
+	/**
+	 * Destroys this instance. Makes sure that all observers are destroyed and listeners removed.
+	 */
+	destroy() {
+		this.roots.map( root => root.destroy() );
+		this.stopListening();
+	}
+
 	/**
 	 * Performs post-fixer loops. Executes post-fixer callbacks as long as none of them has done any changes to the model.
 	 *

src/view/editableelement.js

@@ -77,6 +77,10 @@ export default class EditableElement extends ContainerElement {
 		}
 	}
 
+	destroy() {
+		this.stopListening();
+	}
+
 	/**
 	 * Returns document associated with the editable.
 	 *

src/view/view.js

@@ -398,6 +398,8 @@ export default class View {
 			observer.destroy();
 		}
 
+		this.document.destroy();
+
 		this.stopListening();
 	}