[cla/patch] dijit.WidgetSet - faster toArray() method

[Les]

This simple change results in 2x speed improvement in some browsers.

toArray: function() {
	var ar = [];
	for(id in this._hash){
		arry.push(this._hash[id]);
	}
	return ar;
}

The current code calls an anonymous function on each iteration.

toArray: function(){
	var ar = [];
	this.forEach(function(w){
		ar.push(w); // ^ i, ar[i] = w;
	});
	return ar; // Array
}

[Les]

Fixed typo

toArray: function() {
	var ar = [];
	for(id in this._hash){
		ar.push(this._hash[id]);
	}
	return ar;
}

[bill]

That's true, although I didn't think many people were using toArray(). But I guess it's worth the change. See also #4122.

[Les]

dijit.WidgetSet?.filter() should also be converted to a for loop. This method is used quite often.

[bill]

BTW where is filter() used? I can't think of anyone using it offhand but maybe there are plades.

[Les]

byClass uses filter. I used byClass in the past.

[bill]

(In [20107]) Dijit performance optimizations from Les Szklanny (CLA on file), thanks! Fixes #9885, #9875, #9879, refs #9131. !strict

[bill]

seems like these aren't really enhancements from the users' point of view, labeling as tasks instead