882520 - Two minor tweaks to the memory reporting paths.

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Nicholas Nethercote [inactive]]

I found two minor inconsistencies in the memory reporting paths.

Comment 1

[Nicholas Nethercote [inactive]]

Attachment: Two minor tweaks to the memory reporting paths.

This patch:

- Adds |0x| to the addresses given in some of the "zone(...)" paths.  This is
  necessary for about:memory's diff mode to work -- it strips out 0x-prefixed
  addresses.  And we already have the prefix in some "zone(...)" paths, I just
  forgot to add it in this one case.

- Changes the non-GC lazy script path to use "lazy-script-data" instead of
  "lazy-scripts".  This distinguishes it more clearly from
  "gc-heap/lazy-scripts" and makes it consistent with "script-data".

Comment 2

[Nicholas Nethercote [inactive]]

I fixed a couple more "zone(%p)" instances in dom/workers/WorkerPrivate.cpp, too:

https://hg.mozilla.org/integration/mozilla-inbound/rev/22c88d3d898a

Comment 3

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/22c88d3d898a