Closed
Bug 124053
Opened 23 years ago
Closed 21 years ago
Fix invalid use of JVMContext TLS to map between threads, JSContexts and JSJavaThreadStates
Categories
(Core Graveyard :: Java: OJI, defect)
Core Graveyard
Java: OJI
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: beard, Assigned: beard)
References
Details
Attachments
(2 files)
12.73 KB,
patch
|
bnesse
:
review+
rpotts
:
superreview+
|
Details | Diff | Splinter Review |
245 bytes,
text/html
|
Details |
The OJI module has been playing fast and loose with JSContexts, JSJavaThreadStates and mapping them to the current thread using thread local storage. This is totally bogus, and needs to be cleaned up.
Assignee | ||
Comment 1•23 years ago
|
||
This removes a ton of dead code, and begins to reform the use of the JVMContext thread local storage mechanism.
Comment 2•23 years ago
|
||
Comment 5•23 years ago
|
||
Comment on attachment 68261 [details] [diff] [review] Patch to clean up OJI module >+ JSContext *cx = nsnull; >+ nsCOMPtr<nsIJSContextStack> stack = do_GetService("@mozilla.org/js/xpc/ContextStack;1"); >+ if (stack) stack->Peek(&cx); > > nsCSecurityContext *securityContext = new nsCSecurityContext(cx); Not sure if there is really any point in creating this if cx is null, but it handles the condition internally so I guess it doesn't matter. r=bnesse.
Attachment #68261 -
Flags: review+
Comment 7•22 years ago
|
||
Comment on attachment 68261 [details] [diff] [review] Patch to clean up OJI module sr=rpotts@netscape.com
Attachment #68261 -
Flags: superreview+
Comment 8•21 years ago
|
||
checked in; marking fixed, please reopen if this is not actually fixed Checking in src/jvmmgr.cpp; /cvsroot/mozilla/modules/oji/src/jvmmgr.cpp,v <-- jvmmgr.cpp new revision: 1.42; previous revision: 1.41 done Checking in src/lcglue.cpp; /cvsroot/mozilla/modules/oji/src/lcglue.cpp,v <-- lcglue.cpp new revision: 1.53; previous revision: 1.52 done Checking in src/lcglue.h; /cvsroot/mozilla/modules/oji/src/lcglue.h,v <-- lcglue.h new revision: 1.8; previous revision: 1.7 done
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•