From: Brandon Aaron <brandon.aaron@gmail.com>
Date: Thu, 31 May 2007 00:13:54 +0000 (+0000)
Subject: Use .one() when doing a .bind() with an "unload" event type (#1242)
X-Git-Tag: 1.1.3~28
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d85a22a70ead4a3a22bf1eebe05d08b498f70aa1;p=jquery.git

Use .one() when doing a .bind() with an "unload" event type (#1242)
---

diff --git a/src/event/event.js b/src/event/event.js
index 77a404648..a1a0825dc 100644
--- a/src/event/event.js
+++ b/src/event/event.js
@@ -250,6 +250,9 @@ jQuery.fn.extend({
 	 * data as the second parameter (and the handler function as the third), see 
 	 * second example.
 	 *
+	 * Calling bind with an event type of "unload" will automatically
+	 * use the one method instead of bind to prevent memory leaks.
+	 *
 	 * @example $("p").bind("click", function(){
 	 *   alert( $(this).text() );
 	 * });
@@ -286,7 +289,7 @@ jQuery.fn.extend({
 	 * @cat Events
 	 */
 	bind: function( type, data, fn ) {
-		return this.each(function(){
+		return type == "unload" ? this.one(type, data, fn) : this.each(function(){
 			jQuery.event.add( this, type, fn || data, fn && data );
 		});
 	},