207413 - Midas: designMode = "on" fails for a newly created iframe

Status: RESOLVED FIXED

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

Description

[Ere Maijala (slow)]

I have a simple page with javascript to create an iframe for Midas and set
designMode = "on". The script looks like this: 
  document.writeln("<iframe id="edit"></iframe>");
  document.getElementById("edit").contentDocument.designMode = "on";
The second line doesn't work and an error is logged into JS console. I debugged
it and found out that the document of the iframe doesn't have a mime type yet
when it's checked in nsEditingSession::SetupEditorOnWindow().
nsDocument::StartDocumentLoad() which sets the mime type is called after setting
the designMode and then it's too late. 

Of course there are ways to circumvent this problem, but I think it should work
without any hacks. Jst, do you have any ideas? Is there anything that could be
done to fix this?

Comment 1

[Ere Maijala (slow)]

Attachment: A simple test case

Comment 2

[Johnny Stenback (:jst)]

Attachment: Make nsHTMLDocument always default its content type to text/html.

Comment 3

[Ere Maijala (slow)]

Comment on attachment 124402 [details] [diff] [review]
Make nsHTMLDocument always default its content type to text/html.

Looks good and works great.
r=[e-mail]

Comment 4

[Kathleen :Brade]

-->jst since it's his fix; kin please sr= jst's small change (moving code from
one method to another)

Comment 5

[Mike Kaply [:mkaply]]

Comment on attachment 124402 [details] [diff] [review]
Make nsHTMLDocument always default its content type to text/html.

a=mkaply for 1.4

Comment 6

[Mike Kaply [:mkaply]]

Comment on attachment 124402 [details] [diff] [review]
Make nsHTMLDocument always default its content type to text/html.

a=mkaply for 1.4

Comment 7

[Ere Maijala (slow)]

Fix is checked in to 1.4, waiting for trunk to open.

Comment 8

[Ere Maijala (slow)]

Fixed in trunk too.

Comment 9

[Ninoschka Baca]

Trunk build 2003-06-10: Win2k 
Branch build 2003-06-12: Win2k
I don't see any JS errors using the trunk and branch builds on win2k and the
test cases provided in comment# 1.

Comment 10

[sairuh (rarely reading bugmail)]

thanks for checking this on win2k, Ninoschka.

this also works fine on mac 10.2.6 (2003.06.13.05-1.4) and linux rh7.2
(2003.06.11.07-1.4), commercial branch.

marking verified1.4.