590741 - TM: mozJSComponentLoader needs its own compartment

Status: RESOLVED DUPLICATE of bug 590346

(Core :: JavaScript Engine, defect)

Description

[Gregor Wagner [:gwagner]]

Attachment: full stack

JS_ExecuteScript (cx=0x10686c350, obj=0x11870f3a8, script=0x1068728b0, rval=0x7fff5fbfbd00) at /Users/idefix2/moz/ws3/js/src/jsapi.cpp:4784
4784	    LAST_FRAME_CHECKS(cx, ok);
(gdb) p cx->compartment
$22 = (JSCompartment *) 0x117801a00
(gdb) p cx->runtime->defaultCompartment
$23 = (JSCompartment *) 0x107055400
(gdb) n
4785	    return ok;
(gdb) 
4786	}
(gdb) 
mozJSComponentLoader::GlobalForLocation (this=0x10682cd80, aComponentFile=0x106e02c40, aURI=0x106e02440, aGlobal=0x106e02b90, aLocation=0x106e02b98, exception=0x7fff5fbfc0b0) at /Users/idefix2/moz/ws3/js/src/xpconnect/loader/mozJSComponentLoader.cpp:1512
1512	    *aLocation = ToNewCString(nativePath);
(gdb) p cx->compartment
There is no member or method named compartment.
(gdb) n
1513	    if (!*aLocation) {
(gdb) 
1518	    JS_AddNamedObjectRoot(cx, aGlobal, *aLocation);
(gdb) 
1519	    return NS_OK;
(gdb) 
1520	}
(gdb) 
mozJSComponentLoader::ImportInto (this=0x10682cd80, aLocation=@0x106e01b60, targetObj=0x118707510, cc=0x7fff5fbfca70, _retval=0x7fff5fbfc4c0) at /Users/idefix2/moz/ws3/js/src/xpconnect/loader/mozJSComponentLoader.cpp:1720
1720	        mInProgressImports.Remove(key);
(gdb) 
1722	        if (NS_FAILED(rv)) {
(gdb) 
1738	        mod = newEntry;
(gdb) 
1741	    NS_ASSERTION(mod->global, "Import table contains entry with no global");
(gdb) 
1742	    *_retval = mod->global;
(gdb) 
1745	    if (targetObj) {
(gdb) 
1746	        JSCLContextHelper cxhelper(this);
(gdb) 
1748	        if (!JS_GetProperty(mContext, mod->global,
(gdb) p mContext
$24 = (JSContext *) 0x10686c350
(gdb) p mContext->compartment
$25 = (JSCompartment *) 0x107055400
(gdb) p mContext->runtime->defaultCompartment
$26 = (JSCompartment *) 0x107055400

Comment 1

[Andreas Gal :gal]

Probably fixed.