Ticket #3529: 3529_2.patch

core/dom/text.js

@@ -79 +79 @@
                  */
                 split : function( offset )
                 {
-                        // If the offset is after the last char, IE creates the text node
-                        // on split, but don't include it into the DOM. So, we have to do
-                        // that manually here.
-                        if ( CKEDITOR.env.ie && offset == this.getLength() )
+                        // @see {@link CKEDITOR.env.support.splitTextAtLastChar}.
+                        if ( offset == this.getLength() && !CKEDITOR.env.support.splitTextAtLastChar )
                         {
                                 var next = this.getDocument().createText( '' );
                                 next.insertAfter( this );
@@ -92 +90 @@
                         var doc = this.getDocument();
                         var retval = new CKEDITOR.dom.text( this.$.splitText( offset ), doc );
 
-                        // IE BUG: IE8 does not update the childNodes array in DOM after splitText(),
-                        // we need to make some DOM changes to make it update. (#3436)
-                        if ( CKEDITOR.env.ie && CKEDITOR.env.version >= 8 )
+                        // @see {@link CKEDITOR.env.support.splitTextUpdateChildren}.
+                        if ( !CKEDITOR.env.support.splitTextUpdateChildren )
                         {
                                 var workaround = new CKEDITOR.dom.text( '', doc );
                                 workaround.insertAfter( retval );

core/env.js

@@ -192 +192 @@
                 if ( env.gecko && version < 10900 )
                         env.cssClass += ' cke_browser_gecko18';
 
+                // Feature detections below for capabilities of native API on which we rely.
+                var root = document.documentElement;
+                env.support =
+                {
+                        // IE BUG: If the offset is after the last char, IE creates the text node
+                        // on split, but don't include it into the DOM
+                        splitTextAtLastChar : !env.ie,
+
+                        // IE BUG: IE8 and it's compatible mode does not update the childNodes
+                        // array in DOM after splitText() util next DOM changes.
+                        splitTextUpdateChildren : !( env.ie && env.version == 8 )
+                                                                                && ( function()
+                        {
+                                var parent = document.createElement( 'p' );
+                                parent.innerHTML = 'A B <b>C </b>D E';
+                                root.appendChild( parent );
+                                // Split the text nodes.
+                                parent.firstChild.splitText( 2 );       // Right before "B"
+                                parent.childNodes[ 3 ].splitText( 2 );  // Right before "E"
+                                var childNums = parent.childNodes.length;
+                                root.removeChild( parent );
+                                return childNums == 5;
+                        } )()
+                };
+
                 return env;
         })();
 }