IE: Protected source for comments is broken

[fredck]

The fix for #2263 is breaking source protection in IE SVN nightly.

To replicate:

1. Click source.
2. Add <!-- test --> to the source.
3. Press Source again to revert to design mode.
4. Press Source again, and you have a "null" cannot be null.

[fredck]

The following is a proposal fix for the part of the code that is throwing the error:

But then, there are still other things to get fixed to make it work.

[fredck]

Sorry, Trac didn't like my inline diff... here you have it: http://pastebin.com/f637ed2bf

[martinkou]

Ok... apart from the weird IE bug in which you can't put a bare comment inside a node and expect it to appear in the DOM... there's also the issue that FCKeditor puts protected code as plain text into FCKTempBin, and those plain text should never be converted to DOM nodes at all.

Proposing a patch to fix both issues.

[fredck]

The proposed patch is ok. In my previously proposed patch I got also the chance with this fix to use removeChild to retrieve the final element so we release the reference to the temporary node. Couldn't it be used here too? (basically node.firstChild.removeChild( node.firstChild.lastChild ) at line 1118)

[martinkou]

Fixed with [2258].

Click here for more info about our SVN system.