jquery.extend should set displayName on functions

[dwt]

Both Firebug and the Safari debugger support the convention that if a function has the displayName propperty set, that will be used to display it's name in the debugger.

(see : ​http://www.alertdebugging.com/2009/04/29/building-a-better-javascript-profiler-with-webkit/)

This is mightily usefull as one doesn't get as many 'anonymous functions) in the stacktrace which makes it much easier to work with it.

Here's a monkey-patched version that does enough to make it useful for me - but this should really be in the core of jQuery Extend:

(function() {
	var oldExtend = $.fn.extend;
	
	function isSecondArgumentDictionary(arguments) {
		var methodDict = arguments[1];
		return (2 === arguments.length)
			&& (null !== methodDict || 'object' === typeof(methodDict));
	}
	
	$.extend = function namingExtend() {
		if (isSecondArgumentDictionary(arguments)) {
			var methodDict = arguments[1];
			for (var key in arguments[1]) {
				if ($.isFunction(methodDict[key]) 
					&& ! methodDict[key].displayName)
					methodDict[key].displayName = key;
			}
		}
		return oldExtend.apply(this, arguments);
	};
	
}());

[dwt]

Does this have any chance of getting into core? I gather it is very usefull to all users while not having any incompatibility problems at all.

[snover]

This should go into a debug plugin, not core.