CheckDirty is not returning the correct state

[Senthil]

When i set empty data to the editor using the setdata method, checkdirty is returing the wrong state. Please follow the below steps to see the issue.

1. Open the attached "replacebycode.html" file in IE and click the checkdirty button, this will return the message as 'false'.
2. Now refresh the page and focus the editor area using the mouse cursor and select the checkdirty button, now you will get a message as "true" eventhough we don't change anything inside the editor area.

[garry.yao]

Ticket Test added at : ​http://ckeditor.t/tt/4385/1.html.

[garry.yao]

This's one opposite effect of our auto fixing functionality, where the editor's fixing on DOM structure were confusing the dirty checking mechanism, considering it's changes from the user, which is wrong, I've set up a defensive guard for any future auto fixing rules.

[fredck]

The idea about restoring the dirty status is good. I'm only seeing a drawback in the proposed implementation: performance.

The fact is that the checkDirty call is a bit expensive, specially for long documents. Another fact is that the onSelectionChangeFixBody call is made several times during the document life cycle, even when simply typing inside the editor. These facts together may represent some execution overload.

The onSelectionChangeFixBody function will not always perform document fixes. Actually this should happen rarely. So the implementation could be changed to restore the dirty status only if fixes are supposed to happen to the document. It means that checkDirty should be called only if necessary.

[fredck]

Smart approach ;)

[garry.yao]

Fixed with [4278].

[garry.yao]

Oops, wrong patch used :(
Post-fixing with [4279].