Memory reporters for WebM-related allocations

RESOLVED FIXED in mozilla31

Status

()

defect
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: kinetik, Assigned: froydnj)

Tracking

(Blocks 1 bug)

Trunk
mozilla31
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [MemShrink:P2])

Attachments

(1 attachment)

Reporter

Description

8 years ago
+++ This bug was initially created as a clone of Bug #682215 +++

libnestegg will need to be modified to allow some form of allocator hooking, then the same approach to that used in bug 677653 can be used.
Reporter

Updated

8 years ago
Whiteboard: [MemShrink]
Assignee: nobody → kinetik
Whiteboard: [MemShrink] → [MemShrink:P2]
Reporter

Updated

6 years ago
Assignee: kinetik → nobody
Functional interface for setting libnestegg's allocator posted at https://github.com/kinetiknz/nestegg/pull/19

Once we agree on the interface, we can see about updating the in-tree copy, and then layering memory reporting on top of that.
Reporter

Comment 2

5 years ago
Thanks for the pull request, I've just merged it.  Apologies for the delay; I've been on leave.
Requires bug 992955, but nothing too complicated here.
Attachment #8403328 - Flags: review?(n.nethercote)
Attachment #8403328 - Flags: review?(kinetik)
Reporter

Updated

5 years ago
Attachment #8403328 - Flags: review?(kinetik) → review+
Comment on attachment 8403328 [details] [diff] [review]
add a memory reporter for libnestegg's memory

Review of attachment 8403328 [details] [diff] [review]:
-----------------------------------------------------------------

::: xpcom/build/nsXPComInit.cpp
@@ +572,5 @@
> +
> +    static void* Realloc(void* p, size_t size)
> +    {
> +        sAmount -= MallocSizeOfOnFree(p);
> +        void *pnew = realloc(p, size);

Nit: '*' on the left.
Attachment #8403328 - Flags: review?(n.nethercote) → review+
Commited with changing things to use CountingAllocatorBase.
Assignee: nobody → nfroyd
https://hg.mozilla.org/mozilla-central/rev/5c509bb6f5c4
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.