Domiterator incorrect with enforceRealBlock

[Garry Yao]

When domiterator establish new block for one of the ​pseudo blocks, others should be left untouched, which is currently not respected.

Procedures

1. Open the replace by code example page;
2. Make the following content with selection:

   <div>
   	first
   	<p>paragraph</p>se^cond</div>
   

3. Click on Increase Indent command.

• Actual Result : A new paragraph is establish for text node 'first' and is indented also.

  <div>
  	<p style="margin-left: 40px;">
  		first</p>
  	<p>
  		paragraph</p>
  	<p style="margin-left: 40px;">
  		second</p>
  </div>
  

• Expected Result : The pseudo block contains text 'first' should be left untouched while the 'second' chunk should be indented.

  <div>
  	first
  	<p>
  		paragraph</p>
  	<p style="margin-left: 40px;">
  		second</p>
  </div>
  

[Garry Yao]

Unit Test Case

[Garry Yao]

The patch is mainly about fixing on anonymous block iteration problem, while also introduce a fix to collapsed range boundaries. TCs established for those issues are also in the patch.

[Frederico Caldeira Knabben]

This patch is breaking the getNextSourceNode and getPreviousSourceNode function. If startFromSibling is set to true, it must not in any way take one of the node children as the first node. That's why we have that parameter after all. So it's wrong to default it to the first child. If we have nothing after it, it's correct to have the function returning null.

[Garry Yao]

This one is waiting for #3283.

[Garry Yao]

As discussed no #3203, the getNextSourceNode/getPreviousNode function should not suffer changes, so other implementation should be found to work around this issue.

[Garry Yao]

The patch is containing the latest patch from #3313 for easy reviewing.

[Garry Yao]

The patch updated CKEDITOR.dom.range.enlarge method to reflect the newly introduced walker and fixing the bug on collapsed range; As discovered on #3292, we should not rely on CKEDITOR.dom.range.getBoundaryNodes for collapsed range, the patch instead use temporary bookmark to establish the boundary nodes to simplify codes.

[Garry Yao]

This one will be waiting for #3304.

[Garry Yao]

Unable to reproduce anymore on trunk.