Ticket #4719: 4719_3.patch

_source/core/htmlparser/basicwriter.js

@@ -60 +60 @@
                  */
                 attribute : function( attName, attValue )
                 {
-                        // Browsers don't always escape quote in attribute values. (#4683)
-                        attValue = attValue.replace( /"/g, '"' )
+                        if ( CKEDITOR.env.ie )
+                                attValue = CKEDITOR.tools.htmlEncode( attValue );
 
                         this._.output.push( ' ', attName, '="', attValue, '"' );
                 },

_source/core/tools.js

@@ -316 +316 @@
                                         return fix2( text ).replace( / /g, ' ' );
                                 } :
                                 fix2;
+                                
+                        var fix4 = ( standard( '"' ) == '"') ?
+                                function( text )
+                                {
+                                        // Browsers don't always escape quote in attribute values #4683, #4719.
+                                        return fix3( text ).replace(/"/g, '"');
+                                } :
+                                fix3;
+                                
+                        this.htmlEncode = fix4;
 
-                        this.htmlEncode = fix3;
-
                         return this.htmlEncode( text );
                 },
 

_source/plugins/htmlwriter/plugin.js

@@ -173 +173 @@
                 {
                         if ( this.forceSimpleAmpersand )
                                 attValue = attValue.replace( /&/, '&' );
+                        if ( CKEDITOR.env.ie )
+                                attValue = CKEDITOR.tools.htmlEncode( attValue );
 
-                        // Browsers don't always escape quote in attribute values. (#4683) 
-                        attValue = attValue.replace( /"/g, '"' )
-
                         this._.output.push( ' ', attName, '="', attValue, '"' );
                 },