613912 - Paste list structure from MS-Word conditional comment lost

Status: NEW

(Core :: DOM: Editor, defect)

Description

[Garry Yao]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US) AppleWebKit/534.12 (KHTML, like Gecko) Chrome/9.0.576.0 Safari/534.12
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.12) Gecko/20101026 Firefox/3.6.12

When pasting a list structure from MS Word document on Windows, the conditional comment "<!--[if !supportLists]-->" that was originally presented in older versions has no more appearance.

Reproducible: Always

Steps to Reproduce:
1.Copy the list structure in attached document
2.Paste it into "contentediable" frame
3.Check the HTML codes in 'Source' mode.

Actual Results:  
<p class="MsoNormal" style="margin-left: 18pt; text-indent: -18pt;"><span
style="" lang="EN-US"><span style="">1.<span style="font: 7pt &quot;Times New
Roman&quot;;">&nbsp;&nbsp;&nbsp; </span></span></span><span
dir="LTR"></span><span lang="EN-US">List item1</span></p>

Expected Results:  
<p class="MsoNormal" style="margin-left: 18pt; text-indent: -18pt;"><!--[if !supportLists]--><span
style="" lang="EN-US"><span style="">1.<span style="font: 7pt &quot;Times New
Roman&quot;;">&nbsp;&nbsp;&nbsp; </span></span></span><!--[endif]--><span
dir="LTR"></span><span lang="EN-US">List item1</span></p>

The expected result is what we had in FF3.0

Comment 1

[Garry Yao]

Attachment: MS Word document containing a list structure

Comment 2

[(no longer active)]

Attachment: Clipboard raw HTML data put on the clipboard by Word

This is what Word puts on the clipboard in the HTML format when you copy the list in the attached file to the clipboard.

Note that the conditional comment there is <![if !supportLists]>, which is not a valid HTML comment.

Comment 3

[(no longer active)]

Henri, do you know how the fragment parser handles something like <![if !supportLists]>?  Will it call AddDocTypeDecl on the content sink when it encounters such a thing?  That might explain why we're skipping it, since we just return NS_OK from AddDocTypeDecl without actually doing anything.

Comment 4

[(no longer active)]

Garry, is there any actual reason why these comments are important to you?

Comment 5

[Henri Sivonen (:hsivonen)]

(In reply to comment #3)
> Henri, do you know how the fragment parser handles something like <![if
> !supportLists]>?  Will it call AddDocTypeDecl on the content sink when it
> encounters such a thing?

I don't know what the old parser does. The new parser generates a comment node. Since this is marked post-2.0, I suggest not fixing this in the old parser and getting bug 482909 landed instead.

Comment 6

[(no longer active)]

Makes sense to me.