Grid dojox.grid._View.destroy() leaks nodes/memory in IE

[guest]

Running a simple create/destroy test of the dojox.grid.DataGrid? in sieve shows dom node leakage. I tracked down one source of leakage to the destroy() method in _View.js. I'm not sure if this.headerNode is being properly clobbered.

I looked modified _View.destroy() to use the _Widget way of destroying a domNode. A retest in sieve showed a reduction in the number of dom nodes leaked per create/destroy test.

My modified dojox.grid._View.destroy method is shown below...

destroy: function(){
	//dojox.grid.util.removeNode(this.headerNode);
	dojo._destroyElement(this.headerNode);
	delete this.headerNode;
	this.inherited(arguments);
}

[guest]

More investigation revealed there's a few places where dom nodes are leaked in the DataGrid?. I did manage to eventually track them all down and fix it. I can now create and destroy a DataGrid? at will in Sieve and have zero dom node leakage. I have attached the output of svn diff that shows the required changes. Some of these diffs may not be required, but these changes will result in zero dom node leaks.

Anthony Fryer ([email protected]…)

[Bryan Forbes]

Thanks for your research. I downloaded sIEve ​http://home.wanadoo.nl/jsrosman/sIEve-0.0.8.exe and ran the main test (test_grid.html) before your patch in it and couldn't detect any leaks. I even modified the test to destroy the grid with a button and I still couldn't get it to display any leaks. Your patch looks sound, but I'm not sure it really solves any problems without seeing the results in sIEve myself (not that I don't believe you).

[guest]

modified test_programmatic for node leak sieve testing of grid

[guest]

You can only see widget node leakes if you dynamically create and destroy a widget in a web page. test_grid.html is a once off instantiation of a grid and won't demonstrate node leaks in the grid.

To demonstrate the leak, i've attached a modified test_grid_programmatic.html that provides a button that toggles the grid (ie. creates and destroys the grid). If you run this in Sieve, every time you destroy the grid, you'll notice the DOM Usage increasing. A good widget should return the Dom usage to the same point every time it is destroyed (unless it has form elements but thats an IE issue with forms).

The grid leaks dom nodes because it's not cleaning up objects that reference dom nodes and has dojo.connects without corresponding dojo.disconnects in quite a few places. The patch i provided addressed these issues. I have an updated patch that i'll upload.

Anthony Fryer

[Bryan Forbes]

(In [13900]) fixes #6883 !strict

• Fixed memory leaks in IE.
• Added test to run in sIEve to check for memory leaks

fixes #4983

• Added column reordering for simple views using dojo.dnd.
• Changed column resizing to use dojo.dnd.
• Added test to show column reordering.

[bill]

marking tickets closed in the last three months w/blank milestone to milestone 1.2.