Both IE and Chrome don't have it.

And it's not standard and fun.length is the standard replacement for that.

So took my chances here. - Removed lazyDataProps and defined name the same way as length in resolve - Stop using tinyids (i would like to remove those at some point) - Made arguments/caller behave as expected, previously they wouldn't be resolved from the proto chain

thanks Ms2ger

updated to pass test suite

Comment on attachment 536287 [details] [diff] [review] v2 Review of attachment 536287 [details] [diff] [review]: ----------------------------------------------------------------- Love it. r=me with the two changes below. For future reference -- hg mv is better for renaming a file, since it keeps the file's history intact, and the patch would actually show the diffs for review instead of a whole file being removed and a whole new file being added. ::: js/src/jsfun.cpp @@ +1604,3 @@ > > struct PoisonPillProp { > uint16 atomOffset; Go ahead and get rid of this struct. Make poisonPillProps an array of uint16. Feel free to get rid of it entirely and turn the loop in fun_resolve into a single if-statement, like the length||name one you added. @@ +2834,5 @@ > * parameter, and so binds fun's parent to obj using JSObject::setParent, > * under js_NewFunction (in JSObject::init, called from NewObject -- see > * jsobjinlines.h). > * > + * But JSObject::setParent sets the DELEGATE object flag on its receiver,z Typo!

cdleary-bot mozilla-central merge info: http://hg.mozilla.org/mozilla-central/rev/9ced98ee3aa9

Documentation updated by evilpie: https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/Function/arity https://developer.mozilla.org/en/JavaScript/Reference/Deprecated_Features And mentioned on Firefox 7 for developers.

This was http://hg.mozilla.org/tracemonkey/rev/9ced98ee3aa9

So what did this patch change other than removing .arity? At least one of the bugs that this change caused appears on a web page that doesn't appear to use .arity....

See comment 2

Massive use of arguments in the dropbox zip from bug 666587