Last Comment Bug 722609 - Rename the js-total memory reports
: Rename the js-total memory reports
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: unspecified
: All All
-- normal (vote)
: mozilla13
Assigned To: Nicholas Nethercote [:njn]
: Jason Orendorff [:jorendorff]
Depends on:
  Show dependency treegraph
Reported: 2012-01-30 20:13 PST by Nicholas Nethercote [:njn]
Modified: 2012-02-06 00:54 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (15.24 KB, patch)
2012-01-30 20:13 PST, Nicholas Nethercote [:njn]
luke: review+
Details | Diff | Splinter Review

Description User image Nicholas Nethercote [:njn] 2012-01-30 20:13:34 PST
Created attachment 592977 [details] [diff] [review]

We have various "other" reports in about:memory like "js-total-objects".  These are misnomers, because they only apply to the main JSRuntime, and don't include web worker JSRuntimes.  This patch renames them (e.g. "js-main-runtime-objects").  

I also tree-ified "mjit-code" and "mjit-data" just for the hell of it.  (I've done the same thing with "object-slots" and "object-elements" in bug 722595.)
Comment 1 User image Nicholas Nethercote [:njn] 2012-02-05 18:32:08 PST
Comment 2 User image Marco Bonardo [::mak] 2012-02-06 00:54:08 PST

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