views:

1372

answers:

7

Do you know what may cause memory leaks in JavaScript? I am interested in browsers: IE 7, FireFox 3, Safari 3

A: 

In what browser? Firefox 2? Firefox 3? IE6? IE7? Safari?

Each of them have a different version of the javascript engine. So each of them have their own bug.

Maxim
+2  A: 

In general; circular references are the cause of many problems. I remember IE 6 (not sure if it applies to 7) leaking quite badly with XMLHTTP... setting onreadystatechange = null once it was finished with fixed it.

Greg
Yes the problem applies to 7. And you can't assign null since its not a function you need a void function function() {} that has been created in the global scope and/or assign null to variable holding a reference to the xmlhttp object.
AnthonyWJones
A: 

How are you measuring the leak?

benc
+18  A: 

There is a nice article about JavaScript and memory leaks. It does not specific about on browser, it rather describes the whole problematic of memory leaks and JavaScript.

I think it is a better approach to be as browser unspecific as possible insted of optimizing for a few browsers, when developing a website for the public.

jk
Excellent article on the subject. +1
AnthonyWJones
And for whoever might be interested in IE8, it might be worth to know that most of the example explained in the article are not leaking memory in IE8 anymore. Should read this: http://stackoverflow.com/questions/1999840/javascript-circular-references-and-memory-leaks/2000467#2000467
Marco Demajo
+13  A: 

Here is a classic memory leak in IE:-

function body_onload()
{
    var elem = document.getElementById('someElementId');
    // do stuff with elem
    elem.onclick = function() {
        //Some code that doesn't need the elem variable
    }
 }

After this code has run there is circular reference because an element has a function assigned its onclick event which references a scope object which in turn holds a reference to element.

someElement->onclick->function-scope->elem->someElement

In IE DOM elements are COM based reference counting objects that the Javascript GC can't cleanup.

The addition of a final line in the above code would clean it up:-

var elem = null;
AnthonyWJones
+1  A: 

You can check this MSDN article for Internet Explorer memory leak patterns. Also there are some tools for detecting memory leaks:

korchev
A: 

You're dealing with 2 kinds of objects (and 2 garbage collectors), javascript and DOM objects, which can reference each other (the circular reference), and then neither GC can take care of all its objects even when the page unloads. Here's a good description:

http://getben.com/archive/2006/05/30/Resolving-JavaScript-Memory-Leaks.aspx

http://www.josh-davis.org/2007/04/11/javascript-built-in-listeners-and-memory-leaks/

Gene T