Bug 7012 didn't include support for arrays, because I thought that those are too different between JS and AS. I think I've been misunderstanding it. The attached patch doesn't map circular dependencies (falling back to toString() instead), not sure if that would be desired or not.
Created attachment 10367 [details] proposed patch
Looking good.. one comment. + unsigned numItems = array->getLength(); + for (unsigned i = 0; i < numItems; ++i) + [aeDesc insertDescriptor:aeDescFromJSValue(exec, array->getItem(i)) atIndex:0]; Should this be atIndex:i?
(In reply to comment #2) > Should this be atIndex:i? It could be atIndex:i+1 (AE lists are one-based), but zero also works: "Specifying an index of 0 or count + 1 causes appending to the end of the list." BTW, on a second thought, it seems that it's impossible to create circular dependencies in AE lists anyway, as elements are stored by value, rather than by reference.
Comment on attachment 10367 [details] proposed patch Great, r=me
Committed revision 16200.