============

I had to have some rest... but I am going to put some debug code into UFO... and see what the varibles are set to... to see if I can come up with a fix for you...

http://www.tbshs.herts.sch.uk/o.htm

I have amend ... cleanupIELeaks ... it shows that the inner loop is huge... I get it to jump out at 160 ... to stop system crashing... I am trying to work out how to get the size of this object array at the moment.

cleanupIELeaks: function() {
var _o = document.getElementsByTagName("object");
var _l = _o.length
var cnt = 0;
var totcnt = 0;
var mystr = "";
//alert("MSG 1...");
for (var i = 0; i < _l; i++) {
alert("_o_i is " + _o[i]);
var moink = _o[i];
alert("type is : " + _o[i].type);
alert("typeof is : " + typeof _o[i] );
//alert("sizeof is : " + sizeof(_o[i]) );
//alert("MSG 2...");
_o[i].style.display = "none";
cnt = 0;
for (var x in _o[i]) {
//alert("MSG 3... " + x + " ...");
mystr = mystr + " " + cnt + "] " + x;
cnt++;
totcnt++;
if (totcnt > 180) { alert("_L is " + _l); alert(moink); alert(mystr); return(totcnt); }
if (typeof _o[i][x] == "function") { alert("MSG 4..."); _o[i][x] = null; }
}
}
//
alert("We escaped the loop...");
alert("_L is " + _l);
alert(moink);
alert(mystr);
return(totcnt);
}

Could you please test my workaround inside the leak workaround, I would like to close it before the release, which is expected tomorrow

thanks for the diagnosing of this trouble!!!

Tested it on policy and themes... they now do not crash...

So you can commit that one...

I will keep seeing if there is a better solution in the long run...

where did the original idea for "cleanupIELeaks" come from? Since it crashed both IE6 and IE7... it seems strange that it got in.

I reckon most people did not report it... though being a problem on all IE6 and IE7... because once it crashed... when they reloaded the pages... the changes had already been made to the database... so they were not presented with the policy again... so had no real reason to get annoyed by what they see as a single crash...

you can try to report it upstream, I know very little about this script myself. I guess they should be able to reproduce this quite easily, maybe nobody expected us to be embedding html files with object (for accessibility reasons) instead of iframe.

closing for now

UFO is from http://www.bobbyvandersluis.com/ufo/

I think if I write to the authors... and tell them about our use of object and the problem with UFO... they might suggest that we go back to iframe!

To see how far the loop went... I took out the break and put in window.status = j;

And the loop got upto 50,000 before I stopped it!

I am hoping to get hold of the UFO author and explain the necessary changes... some point next week.