What's the status on this? Didn't you resolve it already? If yes, please set to resolved.

obiweasel,

Can you please verify if this problem persists with 1.91 RC6?

Thanks,

problem persists in 1.91RC6.

Texens,

Still exists with 1.91 RC6

I had the same problem.
This occurs because "document.getElementById('uploader').contentDocument.defaultView" is not defined in IE8 and "document.getElementById('uploader').contentDocument" is not defined in IE6.

I solved this by replacing:

var pass = document.getElementById('uploader').contentDocument.defaultView.saveAndExit(fieldname, show_title, show_comment, pos);

by:

var pass;
if(document.getElementById('uploader').contentDocument)
{
if(document.getElementById('uploader').contentDocument.defaultView)
{
/Firefox/
pass=document.getElementById('uploader').contentDocument.defaultView.saveAndExit(fieldname,show_title,show_comment,pos);
}else{
/IE8/
pass=document.getElementById('uploader').contentWindow.saveAndExit(fieldname,show_title,show_comment,pos);
}
}else{
/IE6/
pass=document.getElementById('uploader').contentWindow.saveAndExit(fieldname,show_title,show_comment,pos);
}

Issue fixed in r10032.

fcabralpacheco,
Thanks a tonne for the patch :)

Looks good,
but why not simply replace all "document.getElementById('uploader').contentDocument.defaultView.saveAndExit" with "document.getElementById('uploader').contentWindow.saveAndExit"?
It works in both IE8 and Firefox I tested, probably no need for the if/else ;?)

yong,

Good idea,
In the recent versions works.
But, I have an application that interacts with the content in an iframe, and in the past had problems with "document.getElementById('*').contentWindow" in older versions of Firefox.