Ticket #7614 (closed defect: fixed)

Opened 10 months ago

Last modified 10 months ago

mail demo fails creating editor in contentpane

Reported by: dante Owned by: doughays
Priority: high Milestone: 1.2
Component: Editor Version: 1.2beta
Severity: normal Keywords:
Cc: bill, liucougar

Description

the nightly version of the mail demo dies in FF3 (and reportedly in IE7) when creating the ContentPane with the href="" editor with:

'contentDoc is undefined'

http://archive.dojotoolkit.org/nightly/checkout/demos/mail/demo.html

Attachments

7614.patch (509 bytes) - added by doughays 10 months ago.
Forces IE7 to follow an existing IE6 path

Change History

Changed 10 months ago by bill

  • owner changed from anonymous to doughays
  • component changed from General to Editor
  • milestone changed from tbd to 1.2

Perhaps related to [14417]. Doug, looks like you didn't merge this change after the refactor?

Changed 10 months ago by peller

  • priority changed from normal to high

Changed 10 months ago by doughays

  • status changed from new to closed
  • resolution set to fixed

(In [15173]) Fixes #7614. References #7097. Remerge Editor change from [14417].

Changed 10 months ago by doughays

  • status changed from closed to reopened
  • resolution deleted

Changed 10 months ago by doughays

  • status changed from reopened to closed
  • resolution set to fixed

(In [15195]) Fixes #7614. References #7097. Remerge Editor change from [14499].

Changed 10 months ago by dante

  • status changed from closed to reopened
  • resolution deleted

still b0rken in ie7

Changed 10 months ago by doughays

  • cc bill, liucougar added

This defect is unrelated to the Editor remerge. It looks like a new timing problem. Accessing iframe.contentWindow in IE7 is throwing an error since its apparently not yet setup completely. Attaching a possible patch but its needs careful review since this is outside my comfort zone.

Changed 10 months ago by doughays

Forces IE7 to follow an existing IE6 path

Changed 10 months ago by bill

Looks good to me but I can't reproduce the error (with or w/out your patch)... I guess that's the nature of race conditions. If it fixes the problem I'd say to check it in.

Changed 10 months ago by doughays

  • status changed from reopened to closed
  • resolution set to fixed

(In [15236]) Fixes #7614. Accessing iframe.contentWindow too early was throwing an Unspecified exception. Used the IE6 code path to delay until the iframe is fully initialized.

Note: See TracTickets for help on using tickets.