'>' replace with '>' on safari and Opera

[Byeongweon Moon]

I found this bug on my google summer of code 2008 period. when I use fckeditor as moinmoin wiki's gui editor, sometimes document parser throw exception. When I insert '>' to editor area it change to '>' after FCKDataProcessor.ConvertToDataFormat executed. as I found, this behavior only happend on apple safari(3.1.1 on window xp. I couldn't test any other safari browser).

You can easily produce this bug using fckeditor's testcase.

1. open _test/automated/tests/fckdataprocessor.html
2. add new testcase end of testcases.

   function test_ConvertToDataFormat_Character_Transform()
   {
   	var dataProcessor = new FCKDataProcessor() ;
   
   	assertEquals(
   		'<p>&lt;&lt;GetText(Edit)&gt;&gt;</p>',
   		dataProcessor.ConvertToDataFormat( document.getElementById('xDiv3'), true, true ) ) ;
   }
   

3. to test this testcase. add below html code inside of body tag

   <div id="xDiv3"><p>&lt;&lt;GetText(Edit)&gt;&gt;</p></div>
   

4. then. run suite on safari.

attached file is testcase what I use and screeanshot of result.

• Browser name and OS : Safari for window (3.1.1) and windows xp professional

[Byeongweon Moon]

testcase file for produce same bug

[Byeongweon Moon]

screenshot of jsunit test result

[Alfonso Martínez de Lizarrondo]

Opera 9.51 also has the same behavior

This seems really a problem in their implementation of XMLSerializer, I'm attaching a simplified testcase that just creates an XML object with such a text node and then tries to serialize it.

[Alfonso Martínez de Lizarrondo]

Simplified testcase

[Alfonso Martínez de Lizarrondo]

Proposed patch

[Alfonso Martínez de Lizarrondo]

According to ​http://www.w3.org/TR/REC-xml/#syntax that output it's correct, although we want all the browsers to have the same behavior, so the proposed patch does include the > in the list of special characters to handle.

I'm attaching also the automated testcase proposed by tasyblue

[Martin Kou]

The patch looks good to me. I think we can have this Review+ed and committed in 2.6.4.

[Alfonso Martínez de Lizarrondo]

Fixed with [2359]