206027 - Cut n paste from Powerpoint into mail compose does not work.

Status: RESOLVED FIXED

(Core :: DOM: Copy & Paste and Drag & Drop, defect)

Description

[Nisheeth Ranjan]

About to attach a patch from Harish that fixes the problem.

Comment 1

[Nisheeth Ranjan]

Attachment: Patch

Fix to htmlparser that initializes the tag type of the CDataSection token. 
This helps the parser construct the right content model when it sees a "<!
blah>" tag in the head of a document.  Powerpoint uses such markup in the HTML
it puts on the clipboard in a cut n paste operation.

Comment 2

[Nisheeth Ranjan]

Comment on attachment 123566 [details] [diff] [review]
Patch

Harish, please r=.  Jst, please sr=.

Please see comment 2 on this bug for more info.

Thanks!

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 123566 [details] [diff] [review]
Patch

sr=jst

Comment 4

[harishd]

Comment on attachment 123566 [details] [diff] [review]
Patch

r=harishd

Comment 5

[Nisheeth Ranjan]

Comment on attachment 123566 [details] [diff] [review]
Patch

Drivers, please approve the checkin of this patch into 1.4.  It makes cut n
paste from Powerpoint into Mozilla work.

Harish came up with this patch to the parser and thinks that it is a low risk
change.  I agree.

Comment 6

[Mike Kaply [:mkaply]]

Comment on attachment 123566 [details] [diff] [review]
Patch

a=mkaply for checkin to to 1.4

Comment 7

[Nisheeth Ranjan]

Checked into trunk:

Checking in public/nsHTMLTokens.h;
/cvsroot/mozilla/htmlparser/public/nsHTMLTokens.h,v  <--  nsHTMLTokens.h
new revision: 1.16; previous revision: 1.15
done
Checking in src/nsDTDUtils.cpp;
/cvsroot/mozilla/htmlparser/src/nsDTDUtils.cpp,v  <--  nsDTDUtils.cpp
new revision: 3.140; previous revision: 3.139
done
Checking in src/nsHTMLTokens.cpp;
/cvsroot/mozilla/htmlparser/src/nsHTMLTokens.cpp,v  <--  nsHTMLTokens.cpp
new revision: 3.234; previous revision: 3.233
done