Unspecified Error when deleting a Row in Internet Explorer 10

[Rick Schnorenberg]

Description

Deleting rows other than the first one lead to an Unspecified Error and cause the UI to be unresponsive for a few seconds.

Steps to Reproduce

1. Launch the Nightly Editor Demo in IE10 (I tested against ​http://nightly.ckeditor.com/13-04-08-14-51/standard/samples/replacebyclass.html)
2. Insert a table using the default options of the dialog
3. Right click on the second or third row and choose Delete Rows
4. An Unspecified Error will be logged in the Developer Tools console and the UI will become unresponsive for a few seconds

System

Browser: Internet Explorer 10 (Desktop Mode)
OS: Windows 8 Enterprise

[Jakub Ś]

Problem can be reproduced in both CKE 4.x (v4) and CKE 3.x. It is reproducible from CKEditor 3.6.6.

Right-clicking on table was possible from rev. [7661]. In that revision this TC has caused "SCRIPT606: Could not complete the operation due to error 800a025e."

Error mentioned in that ticket can be reproduced from rev. [7662].

[Jakub Ś]

Simplified code sample helping to reproduce the problem.

<p><a id="tech-details" name="tech-details"></a></p>

<table border="1" bordercolor="#ccc" cellpadding="5" cellspacing="0" style="border-collapse:collapse; margin:10px 0px 10px 15px">
	<tbody>
		<tr>
			<td>Commander</td>
			<td>Neil A. Armstrong</td>
		</tr>
		<tr>
			<td>Command Module Pilot</td>
			<td>Michael Collins</td>
		</tr>
		<tr>
			<td>Lunar Module Pilot</td>
			<td>Edwin &quot;Buzz&quot; E. Aldrin, Jr.</td>
		</tr>
	</tbody>
</table>

[Jakub Ś]

Another TC:

1. Download attached table.html and copy its HTML into editor.
2. Go to table containing EUR e.g second row, second column.
3. Right-click and select Delete Rows.

You get unspecified error and editor will only start responding when you click on editor document scrollbar.

[Jakub Ś]

Very similar issue was reported in #10894. I would say it is a duplicate that extends this ticket.

[Marek Lewandowski]

Interesting note: it seems to only occur if you don't have a range selected. So if you are moving caret with keyboard etc.

[Marek Lewandowski]

This exception also occured when we attempted to ie.

• remove cell in last ceil of middle row
• remove column

These cases were also included into our test case.

Solution pushed to branch t/10308 at dev and tests.

[Frederico Caldeira Knabben]

Lemme give some background to the proposed fix.

It looks like, in this specific case involving tables, the DOM changes executed by our code are making IE crazy. The selection API for the editable document gets totally broken. By broken I mean: IE start throwing weird errors for anything you try to do with selection without any clear reason.

There is no logic on these errors as we're doing everything right. So the only solution we found for this is "resetting" the selection API, by moving the focus to another document (CKEDITOR.document) and then back to the editable. Fortunately this solved the problem, even if its a damn ugly hack.

[Olek Nowodziński]

Created branch ​t/10308b with the essence of what is in ​t/10308 and does the job.

I also simplified tests by removing obsolete styles, text and stuff, pushed to t/10308@tests-v4.

[Marek Lewandowski]

Fixed with ​git:01b1e52d99a3a (dev) and ae68661b17 (tests) in master.

[Olek Nowodziński]

Moved further discussion to #10960.