557412 - formatblock in contenteditable div with one line inserts blockquote outside div

Status: RESOLVED WORKSFORME

(Core :: DOM: Editor, defect, P5)

Description

[Andrew Hurle [:ahurle]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.2) Gecko/20100316 Firefox/3.6.2 (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.2) Gecko/20100316 Firefox/3.6.2 (.NET CLR 3.5.30729)

If you use document.execCommand('FormatBlock', false, '<blockquote>') in a contenteditable div that only has one line, a blockquote element will be inserted before the editable div.  Will attach test case.

Reproducible: Always

Steps to Reproduce:
1.  Go to the test case page
2.  Click inside the editor to give it focus
3.  Click the blockquote button
Actual Results:  
A blockquote element is inserted outside of the editor, before it (outlined in red).  There is also an error generated in the error console:

Error: Component returned failure code: 0x80004005 (NS_ERROR_FAILURE) [nsIDOMNSHTMLDocument.execCommand]
Source file: http://vestitools.pbworks.com/f/formatblocktest.html
Line: 1


Expected Results:  
Inserted a blockquote element inside the rich text editor

If you type anything on the first line and try to insert a blockquote, it will still be inserted before the editor.  It seems like it can only be prevented (in this example) by inserting a line break.  Then, blockquotes will be entered into the editor as expected.

Comment 1

[Andrew Hurle [:ahurle]]

Attachment: Demonstrates the bug with a contenteditable div and button to execute command

Bad blockquotes are outlined in red, good ones in green

Comment 2

[(mostly gone) XtC4UaLL [:xtc4uall]]

confirmed with 
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a4pre) Gecko/20100405 Firefox/3.7a4pre ID:20100405051240 and 
Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv:1.9.0.19) Gecko/2010031422 Firefox/3.0.19 ID:2010031422

GC 5 dev and Opera 10.5 behave as expected.

unsure if this is already covered by a dependency of Bug 237964.

Comment 3

[Jens Stutte [:jstutte]]

Bulk-downgrade of unassigned, untouched DOM/Storage bug's priority.

If you have reason to believe, this is wrong, please write a comment and ni :jstutte.

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(away until 10/14)]

WFM.

I think that this was fixed by bug 1726064.

Comment 5

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(away until 10/14)]

Perhaps, by this patch.