deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

jQuery.event.fix performance boost. An example: click event was previously ~3ms and is now < 1ms. jQuery.event.trigger also got another small boost in performance.

Browse source
This commit is contained in:
Brandon Aaron 2008-04-28 21:09:27 +00:00
parent 4d652ca520
commit f3f3238c53
1 changed files with 16 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
src/event.js
View file

@ -184,8 +184,16 @@ jQuery.event = {
event = !data[0] || !data[0].preventDefault; event = !data[0] || !data[0].preventDefault;
// Pass along a fake event // Pass along a fake event
if ( event ) if ( event ) {
data.unshift( { type: type, target: elem } ); data.unshift({
type: type,
target: elem,
preventDefault: function(){},
stopPropagation: function(){},
timeStamp: +new Date
});
data[0][expando] = true; // no need to fix fake event
}
// Enforce the right trigger type // Enforce the right trigger type
data[0].type = type; data[0].type = type;
@ -265,11 +273,6 @@ jQuery.event = {
} }
} }
// Clean up added properties in IE to prevent memory leak
if (jQuery.browser.msie)
event.target = event.preventDefault = event.stopPropagation =
event.handler = event.data = event[expando] = null;
return val; return val;
}, },
@ -278,9 +281,12 @@ jQuery.event = {
return event; return event;
// store a copy of the original event object // store a copy of the original event object
// and clone to set read-only properties // and "clone" to set read-only properties
var originalEvent = event; var originalEvent = event;
event = jQuery.extend({}, originalEvent); event = { originalEvent: originalEvent };
var props = "altKey attrChange attrName bubbles button cancelable charCode clientX clientY ctrlKey currentTarget data detail eventPhase fromElement handler keyCode metaKey newValue pageX pageY prevValue relatedNode relatedTarget screenX screenY shiftKey srcElement target timeStamp toElement type view wheelDelta which".split(" ");
for ( var i=props.length; i; i-- )
event[ props[i] ] = originalEvent[ props[i] ];
// Mark it as fixed // Mark it as fixed
event[expando] = true; event[expando] = true;
@ -311,7 +317,7 @@ jQuery.event = {
// check if target is a textnode (safari) // check if target is a textnode (safari)
if ( event.target.nodeType == 3 ) if ( event.target.nodeType == 3 )
event.target = originalEvent.target.parentNode; event.target = event.target.parentNode;
// Add relatedTarget, if necessary // Add relatedTarget, if necessary
if ( !event.relatedTarget && event.fromElement ) if ( !event.relatedTarget && event.fromElement )