Exception occurs after using Replace All and closing Find and Replace dialog in IE11

[Aliaksei]

When i use Replace all feature and close the replace all dialog after that, an exception occurs in IE11.

Environment: OS:Windows 7 x64 Browser: Internet Explorer 11.0.9600.17280 (Update 11.0.12)

Reproducible on demo page: ​http://ckeditor.com/demo#full

Steps:

1. Remove all sample text and images from the CKEditor on the demo page.
2. Type the following text: test text number 1
3. Press Replace button on the toolbar
4. In the Find what field type the following text: number
5. In the Replace with field type the following text: N
6. Press Replace All button
7. Press OK button on the notification Message Box
8. Press close button on the Find and Replace dialog
9. Look in the browser console for the error.

Screen shot is attached.

[Aliaksei]

Exception

[Jakub Ś]

I was able to reproduce this problem in IE9-11 from CKEditor 4.0 (doesn't occur in 4.0 beta).

Error that gets throw is:
Message: SCRIPT5022: IndexSizeError
Line: 1963
URI: selection.js

Similar issues where IndexSizeError was reported are: #12026, #10411. Issues where selection.js error in similar area is : #12212

Similar issues where IndexSizeError and selection.js was reported are: #10858, #12152.

[kkrzton]

Basically when replace dialog is opened in IE, IE loses focus (blur event) so it uses lock selection to save selection range. After replacing the text (with shorter text), the saved ranges are out of date and start/end offset are out of bounds. The error occurs after closing dialog when the selection restoring mechanism tries to restore selection based on outdated ranges.

[kkrzton]

Changes in ​t/12459.

There is also a broader issue with selection restoring #14337.

[Tomasz Jakut]

• TCs can be joined into one file, as there are just two very similar cases.
• TCs are not about wysiwygarea or toolbar plugin (as the tags state) – they are about find plugin and selection. If we are going to put all plugins also as tags, then 95% of our tests would be connected to toolbar plugin ;)
• I'm not sure, but I'd move manual tests to other tests for find plugin.
• It would be nice if we have also unit test(s) for it – mainly because it touches selection.

[kkrzton]

Changes in ​t/12459.

• Manual TCs were merged into one file and moved to find plugin tests dir.
• Tags for TCs were fixed.
• Two unit tests were added.

[Marek Lewandowski]

Moving tickets to the next milestone.

[Marek Lewandowski]

Moving to 4.6.2 minor release, as 4.6.1 is mostly about polishing 4.6.0.

[Marek Lewandowski]

Moving to the nice to have list.