Add JS::MemoryUse values for embedders
Categories
(Core :: JavaScript Engine, enhancement, P2)
Tracking
()
People
(Reporter: ptomato, Assigned: ptomato)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
47 bytes,
text/x-phabricator-request
|
Details | Review | |
1.14 KB,
patch
|
RyanVM
:
approval-mozilla-esr68+
|
Details | Diff | Splinter Review |
The current values in the JS::MemoryUse enum don't necessarily fit the usages that embedders might have for their C-allocated memory. In order to make JS::AddAssociatedMemory() and JS::RemoveAssociatedMemory() more useful for embedders I'd propose adding JS::MemoryUse::Embedding1 through JS::MemoryUse::Embedding5 to the public JS::MemoryUse enum.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
This makes JS::AddAssociatedMemory() and JS::RemoveAssociatedMemory()
more useful for embedders.
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e4f991bcd898
Add JS::MemoryUse::Embedding1 through 5. r=tcampbell,jonco
Assignee | ||
Comment 3•5 years ago
|
||
Comment on attachment 9103724 [details]
Bug 1590845 - Add JS::MemoryUse::Embedding1 through 5. r?tcampbell
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: This makes the JS::AddAssociatedMemory and JS::RemoveAssociatedMemory APIs more useful for SpiderMonkey embedders, who primarily use the ESR version.
- User impact if declined: SpiderMonkey embedders will have less accurate memory tracking in their applications
- Fix Landed on Version: 72
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This adds some more values to an enum, which are not used within Firefox, so it seems quite low-risk.
- String or UUID changes made by this patch: None
Comment 4•5 years ago
|
||
bugherder |
Comment 5•5 years ago
|
||
Comment on attachment 9103724 [details]
Bug 1590845 - Add JS::MemoryUse::Embedding1 through 5. r?tcampbell
This needs a rebased patch for ESR68.
Assignee | ||
Comment 6•5 years ago
|
||
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: I've added a backport of the patch to ESR68. For rationale please see earlier uplift request on this bug.
- User impact if declined:
- Fix Landed on Version:
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky):
- String or UUID changes made by this patch:
Comment 7•5 years ago
|
||
Comment on attachment 9105470 [details] [diff] [review] Backport of patch to ESR68 SM embedding improvements. NPOTB for Firefox. Approved for 68.3esr.
Comment 8•5 years ago
|
||
bugherder uplift |
Description
•