plugin:font change font cross table incorrect

[Garry Yao]

Procedures

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

   ^outter<table><tr><td>text</td></tr></table>outter^
   

3. Apply different fonts three times with the same range.

• Actual Result : The document result in:

  <span style="font-family: Courier New,Courier,monospace;">outter</span>
  <table>
  	<tbody>
  		<tr>
  			<td>
  				<span style="font-family: Courier New,Courier,monospace;">text</span></td>
  		</tr>
  	</tbody>
  </table>
  <table>
  	<tbody>
  		<tr>
  			<td>
  			</td>
  		</tr>
  	</tbody>
  </table>
  <span style="font-family: Courier New,Courier,monospace;">outter</span>
  

[Garry Yao]

I've tested solely with style system which shows no bug, perhaps the bug comes from selection system, just guess.

[Garry Yao]

Unit Test Case

[Garry Yao]

Replying to garry.yao:

I've tested solely with style system which shows no bug, perhaps the bug comes from selection system, just guess.

The bug was actually caused by the style system proved by the attached unit test file.

[Garry Yao]

The problem was due to the invoke to removeFromInsideElement on line 438 will probably remove the startNode which is used as a reference for judging whether an inline element is inside the range.

[Frederico Caldeira Knabben]

This patch breaks several tests at "_source/tests/plugins/styles/styles.html".

I have investigated this issue a lot today and found out that the culprit is a feature I've introduced when porting the style system from V2.

[Garry Yao]

The patch resolve the issue in another simple way.

[Frederico Caldeira Knabben]

Fixed with [3311].