See attachment. Deleting any content directly following a button adds a br "placeholder" to the DOM, which is unexpected. Simply place the caret after the space following the button and hit backspace to repro.
Created attachment 85253 [details] Patch
Comment on attachment 85253 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=85253&action=review looks sane to me. > LayoutTests/editing/deleting/delete-inserts-br-after-button.html:9 > +target = document.getElementById("container"); Nit: Missing "var". > LayoutTests/editing/deleting/delete-inserts-br-after-button.html:11 > +window.getSelection().modify("move", "Right", "character"); Nit: Capitalize / de-capitalize all words here (make them consistent).
(In reply to comment #2) > (From update of attachment 85253 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=85253&action=review > > looks sane to me. > > > LayoutTests/editing/deleting/delete-inserts-br-after-button.html:9 > > +target = document.getElementById("container"); > > Nit: Missing "var". > > > LayoutTests/editing/deleting/delete-inserts-br-after-button.html:11 > > +window.getSelection().modify("move", "Right", "character"); > > Nit: Capitalize / de-capitalize all words here (make them consistent). Thanks for the review! I'll fix those and land :)
Committed r81056: <http://trac.webkit.org/changeset/81056>