Closed
Bug 1797183
Opened 2 years ago
Closed 1 year ago
Use ErrorContext variant of js::ReportOutOfMemory() for JS modules
Categories
(Core :: JavaScript Engine, task, P2)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
109 Branch
Tracking | Status | |
---|---|---|
firefox109 | --- | fixed |
People
(Reporter: bthrall, Assigned: bthrall)
References
Details
Attachments
(1 file)
There are some uses of reportOutOfMemory(JSContext*)
when operating on JS Modules that should be using ErrorContext
because they can be used in a helper thread.
Assignee | ||
Updated•2 years ago
|
Severity: -- → N/A
Priority: -- → P1
Assignee | ||
Comment 1•2 years ago
|
||
Assignee | ||
Updated•2 years ago
|
Priority: P1 → P2
Assignee | ||
Updated•2 years ago
|
Assignee: nobody → bthrall
Updated•2 years ago
|
Attachment #9299992 -
Attachment description: WIP: Bug 1797183 - Use ErrorContext variant of ReportOutOfMemory() → Bug 1797183 - Use ErrorContext variant of ReportOutOfMemory() r=arai
Comment 2•1 year ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:bthrall, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.
Flags: needinfo?(bthrall)
Flags: needinfo?(arai.unmht)
Assignee | ||
Comment 3•1 year ago
|
||
The current patch depends on changes in bug 1782573.
Depends on: 1782573
Flags: needinfo?(bthrall)
Updated•1 year ago
|
Flags: needinfo?(arai.unmht)
Pushed by bthrall@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ceae3867e161 Use ErrorContext variant of ReportOutOfMemory() r=arai
Comment 5•1 year ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox109:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•