CKEDITOR.dom.range::trim() is destroying text ranges.

[Martin Kou]

To reproduce:

1. Open replacebyclass.html.
2. Run the following in Firebug console, or any browser console that allows JavaScript execution:

   var p = CKEDITOR.dom.element.createFromHtml
   ('<p>abcdabcd</p>');
   var range = new CKEDITOR.dom.range();
   range.setStart(p.getFirst(), 2);
   range.setEnd(p.getFirst(), 6);
   console.log(range.collapsed);
   range.trim();
   console.log(range.collapsed);
   

3. range becomes collapsed after running trim() on a non-empty text range, this is wrong.

[Martin Kou]

This bug is caused by endContainer and endOffset being improperly updated after the first part of trim() changed this.endContainer and this.endOffset.

[Garry Yao]

R+ for 3656.patch, appending TC.

[Martin Kou]

Fixed with [3624].

Click here for more info about our SVN system.