Domain relaxation is broken for floating dialogs

[Martin Kou]

This is a regression bug.

To reproduce:

1. Modify sample01.html so that it runs in domain relaxation mode.
2. Open the modified sample01.html.
3. Open a dialog, e.g. the smiley dialog.
4. Permission denied errors.

[Martin Kou]

Another related bug found. If you open and close the image dialog under domain relaxation mode in IE6 will crash the browser.

[Wojciech Olchawa]

Wrong field for Confirmed :)

I will just move the keywords. :)

[Frederico Caldeira Knabben]

As proposed by Alfonso, it seems it is enough to move the domain fixing code at the top of fckdialog.html, before the "var E" declaration.

[Martin Kou]

Replying to fredck:

As proposed by Alfonso, it seems it is enough to move the domain fixing code at the top of fckdialog.html, before the "var E" declaration.

I've done that the first moment I found the bug, but it doesn't fix the IE crash issue.

The IE crash bug is really nasty. It happens with any dialog with a preview pane, i.e. the image dialog, the flash dialog and the image button dialog. The symptom is... you open the dialog, and close it, then IE crashes and would be forced to be closed by Windows. This is a very severe bug as it causes data loss and makes the three dialogs unusable.

The IE crash occurs as long as there is a <script> tag, even an empty one, in the preview pane's HTML. It occurs in IE6 only, and does not occur in IE7.

[Martin Kou]

Proposed patch to fix the permission denied error and the IE6 crash issue. Some other less critical errors have been found for domain relaxation mode in these few days, but those are less critical and they can be fixed after tomorrow's Beta 1 release.

[Frederico Caldeira Knabben]

Please move the dialog crash fix to a dedicated ticket.

Also, for line 194, the following seems to be enough:

setTimeout( function(){ topDialog = null ; }, 0 ) ; 

[Martin Kou]

A new ticket, #1918, has been opened for the IE crash issue.

[Martin Kou]

Fixed with [1611].

Click here for more info about our SVN system.