Merge branch 't/13919'

Author: oleq

CHANGES.md

@@ -24,6 +24,7 @@ Fixed Issues:
 * [#13798](http://dev.ckeditor.com/ticket/13798): [Firefox] Fixed: Changing paragraph format causes the cursor to move.
 * [#13771](http://dev.ckeditor.com/ticket/13771): Fixed: The `contents.css` style is not used if the [IFrame Editing Area](http://ckeditor.com/addon/wysiwygarea) plugin is missing.
 * [#13782](http://dev.ckeditor.com/ticket/13782): Fixed: Unclear log messages.
+* [#13919](http://dev.ckeditor.com/ticket/13919): [Edge] Fixed: Browser window crashes while accessing `isContentEditable` property of an `<input>` DOM element.
 
 Other Changes:
 

core/dom/node.js

@@ -869,8 +869,8 @@ CKEDITOR.tools.extend( CKEDITOR.dom.node.prototype, {
 		if ( this.type != CKEDITOR.NODE_ELEMENT )
 			element = this.getParent();
 
-		// Prevent Edge crash. #13609.
-		if ( CKEDITOR.env.edge && element && element.is( 'textarea' ) ) {
+		// Prevent Edge crash (#13609, #13919).
+		if ( CKEDITOR.env.edge && element && element.is( 'textarea', 'input' ) ) {
 			checkOnlyAttributes = true;
 		}
 

tests/core/dom/node.js

@@ -254,6 +254,26 @@
 			assert.isFalse( el.getChild( [ 0, 0 ] ).isReadOnly( 1 ) );
 		},
 
+		// #13609, #13919
+		'test isReadOnly - isContentEditable property access': function() {
+			// Edge tends to break when accessing isContentEditable property in certain elements.
+			// If this test causes refreshes/crashes the web page, then some new element is causing this issue.
+			var blacklistedElems = {
+					applet: 1 // applet displays a popup about Java at IE11.
+				},
+				elemName;
+
+			// Test every element in DTD.
+			for ( elemName in CKEDITOR.dtd ) {
+				if ( elemName[ 0 ] !== '$' && !( elemName in blacklistedElems ) ) {
+					new CKEDITOR.dom.element( elemName ).isReadOnly();
+				}
+			}
+
+			// If it didn't crash, it's OK.
+			assert.isTrue( true );
+		},
+
 		test_appendTo: function() {
 			var p = newElement( 'p' ),
 				t = newTextNode( 'text' ),