Last Comment Bug 826553 - Add a memory reporter for the event listener manager's hash table.
: Add a memory reporter for the event listener manager's hash table.
Status: RESOLVED FIXED
[MemShrink]
:
Product: Core
Classification: Components
Component: DOM (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla20
Assigned To: Nicholas Nethercote [:njn]
:
:
Mentors:
Depends on:
Blocks: DarkMatter
  Show dependency treegraph
 
Reported: 2013-01-03 17:32 PST by Nicholas Nethercote [:njn]
Modified: 2013-01-04 09:49 PST (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Add a memory reporter for the event listener manager's hash table. (3.88 KB, patch)
2013-01-03 17:32 PST, Nicholas Nethercote [:njn]
bzbarsky: review+
Details | Diff | Splinter Review

Description Nicholas Nethercote [:njn] 2013-01-03 17:32:31 PST
DMD tells me that the event listener manager's hash table can get reasonably
big -- e.g. 384 KiB for me after 8 hours of regular browsing, and 1.5 MiB after
running MemBench.
Comment 1 Nicholas Nethercote [:njn] 2013-01-03 17:32:42 PST
Created attachment 697737 [details] [diff] [review]
Add a memory reporter for the event listener manager's hash table.

This patch adds a reporter for it.  I'm not sure if "explicit/dom/" is the
best path for it;  other suggestions are welcome.
Comment 2 Boris Zbarsky [:bz] (still a bit busy) 2013-01-03 18:34:57 PST
Comment on attachment 697737 [details] [diff] [review]
Add a memory reporter for the event listener manager's hash table.

r=me
Comment 3 Nicholas Nethercote [:njn] 2013-01-03 21:47:06 PST
https://hg.mozilla.org/integration/mozilla-inbound/rev/d7b111082de9
Comment 4 Ed Morley [:emorley] 2013-01-04 09:49:14 PST
https://hg.mozilla.org/mozilla-central/rev/d7b111082de9

Note You need to log in before you can comment on or make changes to this bug.