[Blink/Webkit] Incorrect result of select all, backspace/delete

[Piotrek Koszuliński]

1. Open replacebycode sample.
2. CTRL+A.
3. Delete/backspace.
4. Execute following code in console.

   CKEDITOR.instances.editor1.editable().getHtml();
   

Result:

<h1><small><a data-cke-saved-href="http://en.wikipedia.org/wiki/Apollo_11" href="http://en.wikipedia.org/wiki/Apollo_11"></a></small>​{}</h1>

Expected (as on FF):

<p>[]<br></p>

or (if that's going to be much easier to implement):

<h1>[]<br></h1>

(because <h1> was the first block before pressing backspace)

Related

• #8439, #8440 - should be fixed by patch for this issue,
• this issue was introduced by fixes for #9998.

[Artur Delura]

What I see here is that after pressing delete/backspace editor move whole content from last editable child to first one and empty text content. Based on that I simpified use case:

Having:

<h1>Apollo 11</h1><p><small>Source</small></p>

I above situation first child is <h1>, last one is <p>. After pressing delete/backspace Whole content from <p> has been moved to <h1>. As result we got this:

<h1><small></small>></h1>

[Artur Delura]

Small element has been moved to header by ​this code. Changes and tests in ​branch:t/12503.

[Piotr Jasiun]

On Firefox I get <br> instead of <p><br></p> randomly in manual test, but I met the same behavior on major branch. It occurred randomly and I am not able to reproduce it on console (apparently bluring editor fix issue). Also I was not able to reproduce it with open Firefox DOM inspector, so there is nothing to report.

Closed with ​git:e9db567.

[Piotr Jasiun]

This patch does not change behavior of #8439 and #8440.