Alon Zakai (:azakai) Reporter
	Description • 11 years ago

On Nightly, when I do

1. Load http://www.unrealengine.com/html5/
2. Press run
3. Wait until it finishes starting up (when you are in the market, there is a big popup dialogue, and user input can do stuff), then close the tab
4. Wait, minimize memory use in about memory, etc., then check memory use.

I then see

1,522.21 MB (100.0%) -- explicit
├──1,287.23 MB (84.56%) -- js-non-window
│  ├──1,256.79 MB (82.56%) -- runtime
│  │  ├──1,233.18 MB (81.01%) ── temporary
│  │  └─────23.62 MB (01.55%) ++ (11 tiny)
│  ├─────24.70 MB (01.62%) ++ zones
│  └──────5.74 MB (00.38%) ++ gc-heap

1.2GB is being held onto, even after the tab is closed. Are we leaking all the parse nodes or something like that?

	Andrew McCreight [:mccr8]
	Comment 1 • 11 years ago

"temporary" isn't very descriptive...

	Nicholas Nethercote [inactive]
	Comment 2 • 11 years ago

> "temporary" isn't very descriptive...

But, as usual, the tool-tip in about:memory for "temporary" is descriptive.  This is parse nodes.

	Alon Zakai (:azakai) Reporter
	Comment 3 • 11 years ago

This is almost certainly a regression, because we looked carefully at memory use a few months ago around GDC, and would have seen this.

	Mauricio Collares [:collares]
	Comment 4 • 11 years ago

Works: ftp://ftp.mozilla.org/pub/firefox/tinderbox-builds/mozilla-inbound-linux64/1369916732/
       rev http://hg.mozilla.org/integration/mozilla-inbound/rev/87bbe2a5b08a

Fails: ftp://ftp.mozilla.org/pub/firefox/tinderbox-builds/mozilla-inbound-linux64/1369917031/
       rev http://hg.mozilla.org/integration/mozilla-inbound/rev/d71234d65e90

	Brian Hackett [Laid off!]
	Comment 5 • 11 years ago

Does this still repro after the fix for bug 878293?  That bug could cause mark/release calls on the allocator used for parse nodes to be mismatched, which would break the checks used to immediately release the allocator's memory after a giant parse.

	Mauricio Collares [:collares]
	Comment 6 • 11 years ago

You're right, this no longer happens in today's nightly.