IE8: Error 800a025e is thrown when deleting a row.

[Jakub Ś]

1. Insert 3x3 table.
2. Fill in all cells with data.
3. Go to middle (2x2 cell) and select from context menu "Insert Row Before".
4. Fill in all new cells with data.
5. Select all 3 cells with mouse and then select from context menu "Merge Cells".
6. You will get one row.
7. Right click on this new row and select "Row -> Delete Rows"

Result: in IE8 only error 800a025e is thrown.

Messgae: Error 800a025e
Line: 1542
URI: core/selection.js

Problem can be reproduced in IE8 from CKE 4.2.2.

[Piotr Jasiun]

There is a simpler scenario to get an error:

1. open replacebyclass.html sample,
2. click on "Commander" cell in "Mission crew" table,
3. From the context menu select Row -> Delete Rows.

Result: "'compareDocumentPosition' is null or not an object"

Only deleting last row works properly.

[Olek Nowodziński]

cc

[Piotr Jasiun]

The bug was because of #10308. This hack fixed issue on IE10 but cases issue on IE8 (love IE). It seems to be enough to limit this fix only to IE10, because everything is fine on IE8 & IE9 without this fix.

The reason why tests did not show an error is because you need 4 rows in table (instead of 3) to have a crash removing second row (have I already mentioned that I love IE?). If you have only 3 rows you will get only small error in you console (and not in each case).

Changes in ​t/11058 and corresponding test branch.

[Piotrek Koszuliński]

LovIE!

[Piotr Jasiun]

• ​git:31b7e1f
• test:373209a