Flexigrid - Lightweight but rich data grid

[eluser]pinkhominid[/eluser]
[quote author="pinkhominid" date="1216251009"]Paulo,

I am using flexigrid in a single-page thick-client ajax app that does not do full page postbacks. In this context I have found that flexigrid leaks a pretty significant amount of memory in IE. An instance of the grid with all the bells and whistles (example 3) leaks about 2-5 MB of memory per execution of the flexigrid() method.

Because of the way IE is designed this memory is not freed with a refresh or by closing the tab. It is only freed when all instances of IE are closed.

I've put together a bare minimum demo page that shows the leak. Look at it in IE6 or IE7 and click the 'new instance' buttons multiple times while monitoring IE memory consumption. Each click empties the containing div (using jQuery's empty() method) and creates a new flexigrid instance.

Also, I've tested Firefox & IE8 beta...they DO NOT have the problem.[/quote]

I think I've finally solved the IE memory leak issue. Well...OK...decreased the severity to KBs down from MBs. I had to modify the source a bit to do so. It appears that many of the anonymous functions and some DOM elems were holding onto private vars t, g, and p due to closure. Here is a demo page which implements the fix.

The core change starts at line 1360, replaces 6 lines, and caps off at 30 lines.

Code:

// pinkhominid (2008.09.20): ie leak fix start
        //add document events
        $(document)
        .mousemove(mousemove)
        .mouseup(mouseup)
        .hover(hoverover, hoverout);
        
        function mousemove(e) {
            g.dragMove(e);
        }
        function mouseup() {
            g.dragEnd();
        }
        function hoverover(){}
        function hoverout() {
            g.dragEnd();
        }
        g.cleanup = function () {
            $(document)
                .unbind('mousemove', mousemove)
                .unbind('mouseup', mouseup)
                .unbind('mouseenter mouseleave', hoverout)
                .unbind('mouseenter mouseleave', hoverover);
            p = null;
            g = null;
            t.p = null;
            t.grid = null;
            t = null;
        };
        // pinkhominid (2008.09.20): ie leak fix end

Additionally, I've added a convenience function called flexDestroy in the $.fn plugin section.

Code:

// pinkhominid (2008.09.20): ie leak fix start
    $.fn.flexDestroy = function() {
        return this.each( function() {
                if (this.grid) {
                    $(this.grid.gDiv).remove();
                    this.grid.cleanup();
                }
            });        
    };
    // pinkhominid (2008.09.20): ie leak fix end

Hopefully, this change can be rolled into the next release. Any takers for expanding on this and freeing that last ~150KB that's still leaking?