Creating a dialog leaks memory in IE6

This does not seem to be specific to the dialog. If I create a new widget with no functionality, there is still a leak in IE6.

In the widget factory, if I comment out storage of the new object via "$.data" (lines 253 and 254), then the memory leak does not occur. There is a report in the jQuery bug tracker http://dev.jquery.com/ticket/4375 that indicates a memory leak with $.data when circular references are stored, but I can't yet confirm that as the cause of the problem.

There are actually several memory leaks in dialog when using IE6. They end up leaking the same DOM elements, so it's difficult to tell them apart. This may mean that there is one underlying problem, but I am not yet familiar enough with jQuery to find it. I have been able to find workarounds for every leak exercised by the original example, though I do not necessarily know why they work:

• Applied the patch in #4188 to solve the widget factory leak I mentioned in the previous comment.
• The event bound in disableSelection caused a circular reference of some kind. I added a line to the beginning of dialog.destroy that unbinds it by calling enableSelection:

self.uiDialog.find("*").enableSelection();
 

• I make sure to call destroy for every dialog I create, on unload:

$(window).unload(function()
{
    box.dialog('destroy');
});
 

• I made sure to set draggable to false, as draggable=true caused a leak.

Here is my (supposedly) leak free version of the original code:

	<script type="text/javascript" language="javascript">
		if ($.ui.version != '1.7.1')
		{
			debugger; //This patch may not apply any longer.
		}
		else
		{
			$.ui.dialog.prototype.destroy = function()
			{
				var self = this;

				self.uiDialog.find("*").enableSelection();

				(self.overlay && self.overlay.destroy());
				self.uiDialog.hide();
				self.element
			.unbind('.dialog')
			.removeData('dialog')
			.removeClass('ui-dialog-content ui-widget-content')
			.hide().appendTo('body');

				self.uiDialog.remove();

				(self.originalTitle && self.element.attr('title', self.originalTitle));

				return self;
			}
		}

		$(document).ready(function()
		{
			var box = $('#box');
			var boxOptions = { resizable: false,
				modal: false,
				autoOpen: false
			};

			box.dialog(boxOptions);
			
			$('#msg').text('Working2');

			$(window).unload(function()
			{
				box.dialog('destroy');
			});
		});
	</script>
 

Remember that this is only leak free in conjunction with the patch from #4188.

Ok, this is hilarious. I have been trying to figure out why using jquery ui dialog suddenly causes IE to go from 20kb to 350kb of memory. The reason is the css. Using @ symbols truly F**** IE. Also, if you have a lot of css, it screws it up too.

Duplicate of #4188.