Ticket #3089: 3089_4.patch

_source/tests/core/dom/element.html

@@ -400 +400 @@
                         var element = new CKEDITOR.dom.element( document.getElementById( 'tabIndexScriptDef' ) );
                         assert.areEqual( null, element.getAttribute( 'tabindex' ) );
                 },
+                
+                /**
+                 *  Test getAttribute and getAttribute will ingore  '_cke_expando' attribute. 
+                 */
+                test_getAttribute_ignoreExpandoAttributes : function()
+                {
+                        var element = new CKEDITOR.dom.element( document.getElementById( 'testExpandoAttributes' ) );
+                        assert.areEqual( 1, element.getAttribute( 'tabindex' ) );
+                        element.removeAttribute( 'tabindex' );
+                        assert.areEqual( 'hasExpando', element.getAttribute( 'class' ) );
+                        element.removeAttribute( 'class' );
+                        if( CKEDITOR.env.ie )
+                                assert.areEqual( 'FLOAT: right', element.getAttribute( 'style' ) );
+                        else 
+                                assert.areEqual( 'float: right;', element.getAttribute( 'style' ) );
+                        element.removeAttribute( 'style' );
+                        element.removeAttribute( 'id' );
+                        assert.isFalse( element.hasAttributes() );
+                },
 
                 test_getTabIndex1 : function()
                 {
@@ -505 +524 @@
         <div id="removeClass"></div>
         <div id="removeOuter"><b id="removeInner"></b></div>
         <div id="tabIndex10" tabindex="10"></div>
+        <div id="testExpandoAttributes" _cke_expando ="1" class="hasExpando" tabindex="1" style="float:right"></div>
         <div id="tabIndexDef"></div>
         <input id="tabIndexInputDef" />
         <input id="tabIndexInput20" tabindex="20" />

_source/core/dom/element.js

@@ -397 +397 @@
                                 return this.$.getAttribute( name, 2 );
                         };
 
-                        if ( CKEDITOR.env.ie )
+                        if ( CKEDITOR.env.ie && !CKEDITOR.env.ie8 )
                         {
                                 return function( name )
                                 {
@@ -889 +889 @@
                                 return this;
                         };
 
-                        if ( CKEDITOR.env.ie )
+                        if ( CKEDITOR.env.ie && !CKEDITOR.env.ie8 )
                         {
                                 return function( name, value )
                                 {
@@ -953 +953 @@
                                 this.$.removeAttribute( name );
                         };
 
-                        if ( CKEDITOR.env.ie )
+                        if ( CKEDITOR.env.ie && !CKEDITOR.env.ie8  )
                         {
                                 return function( name )
                                 {

_source/core/env.js

@@ -104 +104 @@
                          *     alert( "I'm on IE6 or quirks mode!" );
                          */
                         env.ie6Compat = ( version < 7 || env.quirks );
+                        
+                        /**
+                         *  Indicate IE8. 
+                         */
+                        env.ie8 = (version >= 8);
                 }
 
                 // Gecko.