[IE] pressing backspace / delete does not fire editor#change

[Marek Lewandowski]

1. open any sample with CKEditor (i.e. samples/replacebyclass.html)
2. add an change listener using console:

   CKEDITOR.instances.editor1.on( 'change', function() { console.log( 'change event' ); } );
   

3. click somewhere in the middle of the text
4. press backspace

Expected result:
Change event should be fired / console log should appear.

Current result:
Change event is not fired.

additional info:

1. tested in IE8 and IE9

[Piotrek Koszuliński]

If it's confirmed please assign it to 4.4.5.

[Jakub Ś]

Reproducible in IE8-11.

It is confirmed and what is more it seems to work in CKEditor 4.4.3 but not on master so it looks like regression.

@Reinmar are you sure about 4.4.5?

[Piotrek Koszuliński]

Yes. It's too late for including it in 4.4.4.

[Marek Lewandowski]

Pushed to ​t/12321 at dev.

Turns out that undoManager#type was not called for IEs when it should. I've also fixed invalid ​comment, since IE is not firing keypress for delete/backspace.

[Olek Nowodziński]

I'm OK with the fix, it looks fine.

Please separate changes in tests from the rest of code before merging into master.

[Piotrek Koszuliński]

Please separate changes in tests from the rest of code before merging into master.

That's not necessary, unless there was some refactorisation. We accept fix+tests in one commit.

[Marek Lewandowski]

Changed milestone to 4.4.4.

Fixed with ​git:67f620b8c2 (merged to master) at dev.