Ticket #3693: 3693.patch

_source/core/dom/element.js

@@ -1 @@
-/*
+/*
 Copyright (c) 2003-2009, CKSource - Frederico Knabben. All rights reserved.
 For licensing, see LICENSE.html or http://ckeditor.com/license
 */
@@ -397 +397 @@
                 {
                         var standard = function( name )
                         {
-                                return this.$.getAttribute( name, 2 );
+                                // Format styles for IE (all up to IE8 strict) and webkit (up to chrome2)
+                                return ( CKEDITOR.env.webkit || CKEDITOR.env.ie ) && name == 'style'
+                                        ? filterStyles( this.$.getAttribute( name, 2 ) )
+                                        : this.$.getAttribute( name, 2 );
                         };
 
+                        var filterStyles = function( styles )
+                        {
+                                return styles
+                                        // 1. Lower case property name.
+                                        // 2. Add space after colon.
+                                        .replace( /(?:^|;)\s*([A-Z-_]+)(:\s*)/ig, function( match, property, colon )
+                                        {
+                                                return property.toLowerCase() + ': ';
+                                        } )
+                                        // 3. Strip whitepsaces around semicolon.
+                                        .replace( /\s+(?:;\s*|$)/g, ';' )
+                                        // 4. Always end with semicolon
+                                        .replace( /([^;])$/g, '$1;'
+                                );
+                        };
+
                         if ( CKEDITOR.env.ie && ( CKEDITOR.env.ie7Compat || CKEDITOR.env.ie6Compat ) )
                         {
                                 return function( name )
@@ -430 +449 @@
 
                                                 case 'style':
                                                         // IE does not return inline styles via getAttribute(). See #2947.
-                                                        var styleText = this.$.style.cssText;
-                                                        return styleText.toLowerCase().replace(
-                                                                /\s*(?:;\s*|$)/, ';').replace(
-                                                                        /([^;])$/, '$1;');
+                                                        return filterStyles( this.$.style.cssText );
                                         }
 
                                         return standard.call( this, name );