Ticket #7614 (closed defect: fixed)

Opened 3 months ago

Last modified 3 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 3 months ago.
Forces IE7 to follow an existing IE6 path

Change History

Changed 3 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 3 months ago by peller

  • priority changed from normal to high

Changed 3 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 3 months ago by doughays

  • status changed from closed to reopened
  • resolution deleted

Changed 3 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 3 months ago by dante

  • status changed from closed to reopened
  • resolution deleted

still b0rken in ie7

Changed 3 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 3 months ago by doughays

Forces IE7 to follow an existing IE6 path

Changed 3 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 3 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.