Skip to main content

Bug Tracker

Side navigation

#3015 closed enhancement (fixed)

Opened June 10, 2008 03:02PM UTC

Closed June 22, 2010 02:41AM UTC

Last modified March 14, 2012 04:51PM UTC

Restrict onunload removal of eventhandlers to IE and Firefox 2

Reported by: JeroenH Owned by:
Priority: minor Milestone: 1.3
Component: event Version: 1.2.6
Keywords: Cc:
Blocked by: Blocking:
Description

In event.js the last lines remove registered eventhandlers on unload in all browsers. This is needed for IE's memory leaks and initially only targeted IE. In #1911 this was changed to apply to all browsers, but the only other browser in need of a fix there was Firefox 2.

This change breaks fast history navigation in Opera (and also in Firefox).

When testing the testcase in #1911, I can reproduce the problem in FF2, but it does not appear anymore in FF3.

It would be better to restrict this change to the browsers that need the fix:

// Prevent memory leaks in IE
// And prevent errors on refresh with events like mouseover in mozilla < 1.9
// Window isn't included so as not to unbind existing unload events
if (jQuery.browser.msie || (jQuery.browser.mozilla && parseFloat(jQuery.browser.version) < 1.9) ){
	jQuery(window).bind("unload", function() {
		jQuery("*").add(document).unbind();
	});
}
Attachments (0)
Change History (5)

Changed June 10, 2008 08:16PM UTC by flesler comment:1

component: coreevent
owner: → brandon
type: bugenhancement

Changed April 22, 2009 01:24AM UTC by brandon comment:2

owner: brandon

Changed September 22, 2009 08:02AM UTC by JeroenH comment:3

This got fixed somewhere, by adding conditional comments for IE.

Changed October 17, 2009 03:23PM UTC by sylvain comment:4

This was fixed in ticket:5198 (changeset:6551).

Changed June 22, 2010 02:41AM UTC by dmethvin comment:5

resolution: → fixed
status: newclosed