Ticket #3240: 3240_4.patch

_source/core/dom/node.js

@@ -424 +424 @@
                         if ( $ == $other )
                                 return CKEDITOR.POSITION_IDENTICAL;
 
-                        // Handle non element nodes (don't support contains nor sourceIndex).
-                        if ( this.type != CKEDITOR.NODE_ELEMENT || otherNode.type != CKEDITOR.NODE_ELEMENT )
+                        // Only element nodes support contains and sourceIndex.
+                        if ( this.type == CKEDITOR.NODE_ELEMENT && otherNode.type == CKEDITOR.NODE_ELEMENT )
                         {
-                                if ( $.parentNode == $other )
-                                        return CKEDITOR.POSITION_IS_CONTAINED + CKEDITOR.POSITION_FOLLOWING;
-                                else if ( $other.parentNode == $ )
-                                        return CKEDITOR.POSITION_CONTAINS + CKEDITOR.POSITION_PRECEDING;
-                                else if ( $.parentNode == $other.parentNode )
-                                        return this.getIndex() < otherNode.getIndex() ? CKEDITOR.POSITION_PRECEDING : CKEDITOR.POSITION_FOLLOWING;
-                                else
+                                if ( $.contains )
                                 {
-                                        $ = $.parentNode;
-                                        $other = $other.parentNode;
+                                        if ( $.contains( $other ) )
+                                                return CKEDITOR.POSITION_CONTAINS + CKEDITOR.POSITION_PRECEDING;
+
+                                        if ( $other.contains( $ ) )
+                                                return CKEDITOR.POSITION_IS_CONTAINED + CKEDITOR.POSITION_FOLLOWING;
                                 }
-                        }
 
-                        if ( $.contains( $other ) )
-                                return CKEDITOR.POSITION_CONTAINS + CKEDITOR.POSITION_PRECEDING;
-
-                        if ( $other.contains( $ ) )
-                                return CKEDITOR.POSITION_IS_CONTAINED + CKEDITOR.POSITION_FOLLOWING;
-
-                        if ( 'sourceIndex' in $ )
-                        {
-                                return ( $.sourceIndex < 0 || $other.sourceIndex < 0 ) ? CKEDITOR.POSITION_DISCONNECTED :
-                                        ( $.sourceIndex < $other.sourceIndex ) ? CKEDITOR.POSITION_PRECEDING :
-                                        CKEDITOR.POSITION_FOLLOWING;
+                                if ( 'sourceIndex' in $ )
+                                {
+                                        return ( $.sourceIndex < 0 || $other.sourceIndex < 0 ) ? CKEDITOR.POSITION_DISCONNECTED :
+                                                ( $.sourceIndex < $other.sourceIndex ) ? CKEDITOR.POSITION_PRECEDING :
+                                                CKEDITOR.POSITION_FOLLOWING;
+                                }
                         }
+                        
+                        // For nodes that don't support compareDocumentPosition, contains
+                        // or sourceIndex, their "address" is compared.
 
-                        // WebKit has no support for sourceIndex.
+                        var addressOfThis = this.getAddress(),
+                                addressOfOther = otherNode.getAddress(),
+                                minLevel = Math.min( addressOfThis.length, addressOfOther.length );
 
-                        var doc = this.getDocument().$;
+                                // Determinate preceed/follow relationship.
+                                for ( var i = 0 ; i <= minLevel - 1 ; i++ )
+                                {
+                                        if ( addressOfThis[ i ] != addressOfOther[ i ] )
+                                        {
+                                                if ( i < minLevel )
+                                                {
+                                                        return addressOfThis[ i ] < addressOfOther[ i ] ?
+                                                            CKEDITOR.POSITION_PRECEDING : CKEDITOR.POSITION_FOLLOWING;
+                                                }
+                                                break;
+                                        }
+                                }
 
-                        var range1 = doc.createRange();
-                        var range2 = doc.createRange();
-
-                        range1.selectNode( $ );
-                        range2.selectNode( $other );
-
-                        return range1.compareBoundaryPoints( 1, range2 ) > 0 ?
-                                CKEDITOR.POSITION_FOLLOWING :
-                                CKEDITOR.POSITION_PRECEDING;
+                                // Determinate contains/contained relationship.
+                                return ( addressOfThis.length < addressOfOther.length ) ?
+                                                        CKEDITOR.POSITION_CONTAINS + CKEDITOR.POSITION_PRECEDING :
+                                                        CKEDITOR.POSITION_IS_CONTAINED + CKEDITOR.POSITION_FOLLOWING;
                 },
 
                 /**

_source/tests/core/dom/node.html

@@ -9 +9 @@
         <script type="text/javascript" src="../../test.js"></script>
         <script type="text/javascript">
         //<![CDATA[
-
-CKEDITOR.test.addTestCase( (function()
+var tc;
+CKEDITOR.test.addTestCase( tc = (function()
 {
         // Local reference to the "assert" object.
         var assert = CKEDITOR.test.assert;
@@ -68 +68 @@
                 {
                         var node1 = new CKEDITOR.dom.node( document.getElementsByTagName( 'h1' )[0] );
                         var node2 = new CKEDITOR.dom.node( document.getElementsByTagName( 'h1' )[0].firstChild );
-
                         assert.areSame( CKEDITOR.POSITION_IS_CONTAINED + CKEDITOR.POSITION_FOLLOWING, node2.getPosition( node1 ) );
                 },
 
@@ -104 +103 @@
                         assert.areSame( CKEDITOR.POSITION_FOLLOWING, node2.getPosition( node1 ) );
                 },
 
+                /**
+                 *  Test 'preceding' position.
+                 */
+                test_getPosition_3240: function()
+                {
+                        var node1 = new CKEDITOR.dom.node( document.getElementsByTagName( 'b' )[0].firstChild );
+                        var node2 = new CKEDITOR.dom.node( document.getElementsByTagName( 'span' )[0].firstChild );
+
+                        assert.areSame( CKEDITOR.POSITION_PRECEDING, node1.getPosition( node2 ) );
+                },
+
                 name : document.title
         };
 })() );
@@ -115 +125 @@
 //
 //      alert( node1.getPosition( node2 ) );
 //};
-
+//window.onload =  tc.test_getPosition6;
         //]]>
         </script>
 </head>
@@ -124 +134 @@
                 <h1>Title</h1>
                 <p><b>Sample</b> <i>Text</i></p>
         </div>
+        <span>Another</span>
 </body>
 </html>