Ticket #3529: 3529.patch

_source/core/dom/text.js

@@ -77 +77 @@
                  * @param {Number} The position at which to split, starting from zero.
                  * @returns {CKEDITOR.dom.text} The new text node.
                  */
-                split : function( offset )
+                split : ( function()
                 {
-                        // 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() )
-                        {
-                                var next = this.getDocument().createText( '' );
-                                next.insertAfter( this );
-                                return next;
-                        }
+                        function standard( 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())
+                                {
+                                        var next = this.getDocument().createText('');
+                                        next.insertAfter(this);
+                                        return next;
+                                }
 
-                        var doc = this.getDocument();
-                        var retval = new CKEDITOR.dom.text( this.$.splitText( offset ), doc );
+                                var doc = this.getDocument();
+                                return 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 )
-                        {
-                                var workaround = new CKEDITOR.dom.text( '', doc );
-                                workaround.insertAfter( retval );
-                                workaround.remove();
-                        }
-
-                        return retval;
-                },
+                        // IE8 is buggy with splitText.
+                        function supportSplitText()
+                        {
+                                return Object.prototype.toString.call( self.JSON )  !=  "[object JSON]";
+                        }
+
+                        if ( supportSplitText() )
+                                return standard;
+                        else
+                                return function( offset )
+                                {
+                                        var retval = standard.call( this, offset );
+                                        // 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)
+                                        var workaround = this.getDocument().createText( '' );
+                                        workaround.insertAfter( retval );
+                                        workaround.remove();
+                                        return retval;
+                                };
+                } )(),
 
                 /**
                  * Extracts characters from indexA up to but not including indexB.