Bug 118457 – DeleteSelectionCommand does not handle correctly deletion with two consecutive BR element.

• Status: RESOLVED
• Resolution: FIXED
• Component: HTML Editing

[Enrica Casucci]

Created attachment 206220 [details]
Test case

Repro steps:

Open the attached test case (test2br.html).
Click on the empty line below the empty image box
Press backspace.

Expected:
the empty line is removed and the caret moves after the image.

Actual:
Nothing happens. The line is not removed and the caret remains in the same position.

<rdar://problem/14160912>

[Enrica Casucci]

Created attachment 206224 [details]
Patch

[Simon Fraser (smfr)]

Comment on attachment 206224 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=206224&action=review

> Source/WebCore/editing/DeleteSelectionCommand.cpp:312
> +    // when the two elements are sibling and should be false in a case like <div><br></div><br>.

…are siblings...

> LayoutTests/editing/deleting/delete-image-followed-by-two-br-expected.txt:1
> +CONSOLE MESSAGE: line 16: TypeError: 'undefined' is not a function (evaluating 'document.getElementsById("test")')

Should this be here?

[Enrica Casucci]

(In reply to comment #2)
> (From update of attachment 206224 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=206224&action=review
> 
> > Source/WebCore/editing/DeleteSelectionCommand.cpp:312
> > +    // when the two elements are sibling and should be false in a case like <div><br></div><br>.
> 
> …are siblings...
Fixed.
> 
> > LayoutTests/editing/deleting/delete-image-followed-by-two-br-expected.txt:1
> > +CONSOLE MESSAGE: line 16: TypeError: 'undefined' is not a function (evaluating 'document.getElementsById("test")')
> 
> Should this be here?

No, it is a syntax error. Extra s in getElementById.
thanks for catching it.

[Enrica Casucci]

Committed revision 152478.