Fixed a bunch of endline issues.

[jquery.git] / src / event / event.js
diff --git a/src/event/event.js b/src/event/event.js

index ff1d936..86a93d4 100644 (file)
--- a/src/event/event.js
+++ b/src/event/event.js
@@ -80,7 +80,7 @@ jQuery.fn.extend({
                         var p = (e.type == "mouseover" ? e.fromElement : e.toElement) || e.relatedTarget;
         
                         // Traverse up the tree
-                       while ( p && p != this ) p = p.parentNode;
+                       while ( p && p != this ) try { p = p.parentNode } catch(e) { p = this; };
                         
                         // If we actually just moused on to a sub-element, ignore it
                         if ( p == this ) return false;
@@ -107,6 +107,8 @@ jQuery.fn.extend({
          * Please ensure you have no code in your &lt;body&gt; onload event handler, 
          * otherwise $(document).ready() may not fire.
          *
+        * You can have as many $(document).ready events on your page as you like.
+        *
          * @example $(document).ready(function(){ Your code here... });
          *
          * @name ready
@@ -153,6 +155,9 @@ jQuery.extend({
                                 // Reset the list of functions
                                 jQuery.readyList = null;
                         }
+                       // Remove event lisenter to avoid memory leak
+                       if ( jQuery.browser.mozilla || jQuery.browser.opera )
+                               document.removeEventListener( "DOMContentLoaded", jQuery.ready, false );
                 }
         }
  });
@@ -720,11 +725,15 @@ new function(){
                  * Trigger the load event of each matched element. This causes all of the functions
                  * that have been bound to thet load event to be executed.
                  *
+                * Marked as private: Calling load() without arguments throws exception because the ajax load
+                * does not handle it.
+                *
                  * @example $("p").load();
                  * @before <p onload="alert('Hello');">Hello</p>
                  * @result alert('Hello');
                  *
                  * @name load
+                * @private
                  * @type jQuery
                  * @cat Events/Browser
                  */
@@ -1521,7 +1530,8 @@ new function(){
                  
                  /**
                   * @test var count;
-                 * var e = ("blur,focus,load,resize,scroll,unload,click,dblclick," +
+                 * // ignore load
+                 * var e = ("blur,focus,resize,scroll,unload,click,dblclick," +
                   *             "mousedown,mouseup,mousemove,mouseover,mouseout,change,reset,select," + 
                   *             "submit,keydown,keypress,keyup,error").split(",");
                   * var handler1 = function(event) {
@@ -1554,7 +1564,7 @@ new function(){
                   *     $(document)[event]();
                   *     
                   *     // assert count
-                 * @test ok( count == 6, 'Checking event ' + event);
+                 *     ok( count == 6, 'Checking event ' + event);
                   * }
                   *
                   * @private
@@ -1646,3 +1656,12 @@ new function(){
         jQuery.event.add( window, "load", jQuery.ready );
         
  };
+
+// Clean up after IE to avoid memory leaks
+if (jQuery.browser.msie) jQuery(window).unload(function() {
+       var event = jQuery.event, global = event.global;
+       for (var type in global) {
+               var els = global[type], i = els.length;
+               if (i>0) do if (type != 'unload') event.remove(els[i-1], type); while (--i);
+       }
+});