bugzilla.mozilla.org will be intermittently unavailable on Saturday, March 24th, from 16:00 until 20:00 UTC.



Audio/Video: Playback
2 years ago
2 years ago


(Reporter: Jesse Ruderman, Unassigned, NeedInfo)


(Blocks: 1 bug, {memory-leak, testcase})

memory-leak, testcase

Firefox Tracking Flags

(firefox45 affected)


(Whiteboard: [MemShrink:P2])


(1 attachment)



2 years ago
Created attachment 8687193 [details]
testcase (requires extension)

Leaks MediaMemoryTracker and nsTArray_base, as reported by trace-refcnt.

The testcase calls fuzzPriv.getMemoryReports, which does pretty much the same calls as clicking "Measure" in about:memory, but lets the testcase control the timing. Run with https://github.com/MozillaSecurity/funfuzz/tree/master/dom/extension installed in a temporary profile. (Or modify the testcase so the timing doesn't need to be so precise?)
Component: Audio/Video → Audio/Video: Playback
You you interested in this one, Gerald?
Flags: needinfo?(gsquelart)
Priority: -- → P2
Whiteboard: [MemShrink]
It looks like we expect the MediaMemoryTracker to be cleaned up when we remove the last MediaDecoder [1]. So presumably that's not happening.

Jesse, are these the STR? Are you using a special build / env flags?
1 - Load testcase (I'm assuming this could probably just be a xpcshell test)
2 - Quit

[1] https://dxr.mozilla.org/mozilla-central/rev/a2f83cbe53ac4009afa4cb2b0b8f549289b23eeb/dom/media/MediaDecoder.cpp#103-111
Flags: needinfo?(jruderman)
(Just removing my NI, will wait for answers to Eric's questions)
Flags: needinfo?(gsquelart)

Comment 4

2 years ago
1. Make a fresh profile with e10s disabled.
2. Install https://github.com/MozillaSecurity/funfuzz/tree/master/dom/extension/

     A. Set xpinstall.signatures.required to false
     B. wget https://www.squarefree.com/extensions/domFuzzLite3.xpi
     C. Drag xpi to Firefox

     A. Clone https://github.com/MozillaSecurity/funfuzz/
     B. Set a bunch of prefs
     C. Make a text file link

3. XPCOM_MEM_LEAK_LOG=2 firefox -profile <dir> q.html
4. Quit
Flags: needinfo?(jruderman)
(In reply to Gerald Squelart [:gerald] from comment #3)
> (Just removing my NI, will wait for answers to Eric's questions)

Gerald, we have STR now (and I confirmed they worked).
Flags: needinfo?(gsquelart)
Whiteboard: [MemShrink] → [MemShrink:P2]
Still in my to-do list, keeping NI as reminder.
Mass change P2 -> P3
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.