Merge branch 't/10219b'

Author: oleq

CHANGES.md

@@ -4,6 +4,7 @@ CKEditor 4 Changelog
 ## CKEditor 4.1.1
 
 * [#10172](http://dev.ckeditor.com/ticket/10172): Pressing delete/backspace in empty table cell moves cursor to next/previous cell.
+* [#10219](http://dev.ckeditor.com/ticket/10219): Error thrown when destroying an instance in parallel with a mouseup event.
 
 ## CKEditor 4.1
 

plugins/clipboard/plugin.js

@@ -477,15 +477,24 @@
 				!preventBeforePasteEvent && fixCut( editor );
 			});
 
+			var mouseupTimeout;
+
 			// Use editor.document instead of editable in non-IEs for observing mouseup
 			// since editable won't fire the event if selection process started within
 			// iframe and ended out of the editor (#9851).
 			editable.attachListener( CKEDITOR.env.ie ? editable : editor.document.getDocumentElement(), 'mouseup', function() {
-				setTimeout( function() {
+				mouseupTimeout = setTimeout( function() {
 					setToolbarStates();
 				}, 0 );
 			});
 
+			// Make sure that deferred mouseup callback isn't executed after editor instance
+			// had been destroyed. This may happen when editor.destroy() is called in parallel
+			// with mouseup event (i.e. a button with onclick callback) (#10219).
+			editor.on( 'destroy', function() {
+				clearTimeout( mouseupTimeout );
+			});
+
 			editable.on( 'keyup', setToolbarStates );
 		}