Closed
Bug 1255874
Opened 9 years ago
Closed 8 years ago
Get rid of GetScriptContextFromJSContext
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox48 | --- | affected |
People
(Reporter: bzbarsky, Unassigned)
References
Details
(Whiteboard: btpp-backlog)
No description provided.
Reporter | ||
Comment 1•9 years ago
|
||
This is used in a few places:
1) The window-closing-blocking thing in nsGlobalWindow::OpenInternal. Seems like this could use the entry global instead (comparing it to "this").
2) CycleCollectedJSRuntime::UsefulToMergeZones. We need to rewrite this anyway for the one-JSContext world.
3) ScriptSettings, to keep the JSContext alive in the AutoCxPusher. Won't be an issue in the one-JSContext world.
4) GetDefaultScopeFromJSContext. This is only used in XPCJSContextStack::Push.
So I guess once we fix #1 we'll be blocked on the jscontext stack removal to do this....
Updated•9 years ago
|
Whiteboard: btpp-backlog
Reporter | ||
Updated•9 years ago
|
Reporter | ||
Comment 2•8 years ago
|
||
The JSContext stack bit will go away in bug 1276276, I think.
Depends on: 1276276
Reporter | ||
Comment 3•8 years ago
|
||
This is now fixed, as of bug 1276276 landing.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•