Use displayName to help debugging dojo.declare in webkit

[Adam Peller]

From Patrick Mueller (IBM):

try adding the following line in safeMixin() after the lines:

	t.nom = name; target[name] = t; 

The line to add:

	if (typeof t == "function" && !t.displayName) t.displayName = name;

Here's something I just added to d.declare() that seemed to work; added it before the safeMixin() call; I ended up seeing things like "demo.Teacher::blink" in my call stack where before I just saw "(anonymous function)":

		for (var propName in props) {
			if (props.hasOwnProperty(propName)) {
				var propVal = props[propName]
				if (typeof propVal == "function") {
					if (!propVal.displayName) {
						propVal.displayName = className + "::" + propName
					}
				}
			}
		}

[Eugene Lazutkin]

The idea is good, but let's keep the base size small. I think we need to create a core module (dojo.debug?) to keep those helpers as addons. When e this module is included, it will decorate all dojo.declare-produced prototypes accordingly.

Rescheduling to 1.6 for now.

[Eugene Lazutkin]

It looks like Webkit changed the way it shows method names, so the number of anonymous functions are greatly reduced. Closing this ticket for now.

[pmuellr]

I'm fine with closing the ticket, but thought I'd chime in.

It's not Webkit that changed, but V8. JSC, AFAIK, does not infer function names the way V8 does. So you will see more named functions in Chrome's DevTools? than you will in Safari Web Inspector, for instance.