following code in chrome context crashes mozilla:
Yeah ... don't do that ;-)
Or we could NS_ENSURE_ARG_POINTER and be done with it. Might be worth it....
Not so fun; bug 587698 disabled *all* our event listener service tests, and now
even those ones not using JSD fail because of compartment mismatch, and once that problem is fixed
there is an assertion I don't quite understand yet...
Created attachment 565801 [details] [diff] [review]
but getEventTargetChainFor call causes still
###!!! ASSERTION: Non-global object has the wrong flags: '!(jsclazz->flags & JSCLASS_IS_GLOBAL)', file /home/smaug/mozilla/hg/m-c/js/src/xpconnect/src/xpcwrappednative.cpp, line 1146
XPCWrappedNative::Init(XPCCallContext&, JSObject*, int, XPCNativeScriptableCreateInfo const*) (/home/smaug/mozilla/hg/m-c/js/src/xpconnect/src/xpcwrappednative.cpp:1145)
So the assertion happens somehow when page's JS touches TabChildGlobal.
I wonder what has changed since bug 587698 to cause that assertion to fire.
(The assertion has been there for a long time)
http://mxr.mozilla.org/mozilla-central/source/js/src/xpconnect/src/xpcconvert.cpp#486 is in the stack..
Ok, found the problem, patch coming.
Created attachment 565809 [details] [diff] [review]
The patch has sort of random stuff that is needed to be able to run the
tests again without errors or assertions.
Comment on attachment 565809 [details] [diff] [review]