Mihai Sucan [:msucan] Reporter
	Description • 12 years ago

For the window.console API implementation (dom/base/ConsoleAPI.js) we have to walk the Components.stack frames, from caller to caller to find the frames that are in JS and build an array.

We use Components.stack for every window.console API call to be able to show a stack trace in the Web Console for each logged message.

While working on fixing bug 746869 we've found that walking Components.stack is quite slow. We'd appreciate some performance improvements.

Thanks!

	Boris Zbarsky [:bzbarsky]
	Comment 1 • 12 years ago

The basic problem is that the nsIStackFrames and whatnot are using full-on XPConnect wrappers... and are REALLY SLOW (tm).

There's the JS-wrapping being slow, the lookup of the methods, the security checks, etc etc.

Oh, and the methods don't live on the proto, so for each stackframe we define it on the object.

I wonder whether we could move stackframes to some saner setup.  Somehow.

	Bobby Holley (:bholley)
	Comment 2 • 12 years ago

A simple measure might be to define nsIClassInfo, which would let the methods live on the proto. Is that known to be faster?

	Boris Zbarsky [:bzbarsky]
	Comment 3 • 12 years ago

Well, my profile shows 20% of the time is DefinePropertyIfFound, iirc.

Simply trying out classinfo should tell us whether things got 205 faster....