implicitly declared global variables and unused local variables

[Simon Speich]

I used jetbrains phpStorm's code inspection to analyze dojo and dijit and found the following problematic code in trunk:

--- implicitly declared global variables ---
dijit.Tree.js line 1861:			nodes = [];
dom-attr.js line 83:				value = node[propName];

--- unused local variables ---
dijit.tree._dndContainer.js line 74:		var self = this;
dijit.form._SearchMixin.js line 257:		var srcNodeRef = this.srcNodeRef;
dijit.layout.StackController.js line 95:	page = button.page;
dojo.dnd.Avatar.js line 43:			icon = has("highcontrast") ? ...
dojo.js line 881:				var ..., path, ...
dojo.js line 1076:				var pseudoMid = pseudoPluginResource.mid,
dom-geometry.js line 529:			dh = db.parentNode,
has.js line line 106:				var agent = navigator.userAgent;
dojo.io.script.js line 46:			var loadEvent = has("ie") ? "onreadystatechange" : ...
dojo.io.script.js line 47:			readyRegExp = /complete|loaded/;
OpenAjax.js line 29/30/31:			var t; var f; var g;
dojo.selector.acme.js line 1107:		var nua = navigator.userAgent;

[bill]

Thanks, I'll fix.

[bill]

In [29096]:

Fix unintended globals and unused variables, thanks Simon, fixes #15592 !strict.