Insert table in empty document add extra line

[Garry Yao]

Reproducing Procedures

1. Open the replace by class example page in FF;
2. Click on 'New Page' to empty the document;
3. Insert a default table;
   • Expected Result:

     <table border="1" cellpadding="1" cellspacing="1" style="width: 200px;">
     ...
     </table>
     <p>
     	<br />
     </p>
     

   • Actual Result: There's an paragraph established before the table:

     <p>
     	<br />
     </p>
     <table border="1" cellpadding="1" cellspacing="1" style="width: 200px;">
     ...
     </table>
     <p>
     	<br />
     </p>
     

[Garry Yao]

Not sure if we need to trim the last paragraph, so keep it in the expected result.

[Artur Formella]

With this patch it is not possible to insert anything above the table.

[Frederico Caldeira Knabben]

The right way to check if we're in an empty block is by checking if the range is checkStartOfBlock "and" checkEndOfBlock.

[Garry Yao]

Ticket Test added at : ​http://ckeditor.t/tt/3664/1.html.

[Garry Yao]

Replying to arczi:

With this patch it is not possible to insert anything above the table.

The exitable table feature is a separate topic and should be handled by other tickets, the patch is about that.

[Frederico Caldeira Knabben]

• The "children", "count", "firstChild" are still in the code (not used).

• The "if" declaration at line 85 is missing the curly braces.

[Garry Yao]

Fixed with [4344].