Closed
Bug 540981
Opened 15 years ago
Closed 13 years ago
push data from JSContext down to js::StackSegment
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: luke, Unassigned)
References
Details
JS_{Save,Restore}FrameChain allow a single JSContext to switch in the middle of executing one JS callstack to start executing a second callstack. This is currently achieved by simply saving the original cx->fp and restoring it later. This subtle reuse of JSContext can lead to unlikely and subtle bugs where data stored in JSContext is associated with a single callstack. For example, 'throwing' and 'exception' are associated with a single callstack and need to be handled specially to avoid leaking between frame chains. It also seems possible for 'resolvingTable' and 'busyArrayTable' to leak with visible side effects.
With bug 535656, there is a JSCallStack data structure associated with a single segment of a callstack that may be a more approriate place for this and future data. This bug proposes moving callstack-related data from JSContext to JSCallStack. Other candidates for moving include: 'regExpStatics', 'sharpObjectMap', 'lastMessage', and 'display'.
Reporter | ||
Updated•14 years ago
|
Summary: push data from JSContext down to JSCallStack → push data from JSContext down to js::StackSegment
Reporter | ||
Comment 1•13 years ago
|
||
How about just removing JSContext altogether (bug 650351).
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Reporter | ||
Comment 2•13 years ago
|
||
Er, bug 650361.
You need to log in
before you can comment on or make changes to this bug.
Description
•