Closed
Bug 1223691
Opened 9 years ago
Closed 5 years ago
MediaStreamGraph leak
Categories
(Core :: Web Audio, defect, P4)
Core
Web Audio
Tracking
()
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
firefox42 | --- | unaffected |
firefox43 | --- | wontfix |
firefox44 | --- | wontfix |
firefox45 | --- | wontfix |
People
(Reporter: jruderman, Unassigned)
References
Details
(Keywords: memory-leak, regression, testcase, Whiteboard: [MemShrink:P2])
Attachments
(2 files)
1. Run with XPCOM_MEM_LEAK_LOG=2 2. Load the testcase 3. Quit Firefox Result: leak (Tested with e10s disabled)
Reporter | ||
Comment 1•9 years ago
|
||
Reporter | ||
Updated•9 years ago
|
Attachment #8685867 -
Attachment description: stack → leak log from trace-refcnt
Comment 2•9 years ago
|
||
An AudioContext will keep the graph alive, so I'll move this to Web Audio. However, bug 1200514 is not fixed and is likely to cause leaks such as this.
Component: Audio/Video: MediaStreamGraph → Web Audio
Depends on: 1200514
Updated•9 years ago
|
Keywords: regressionwindow-wanted
Updated•9 years ago
|
Whiteboard: [MemShrink]
Updated•9 years ago
|
Priority: -- → P3
Comment 3•9 years ago
|
||
Regression range: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=f78347b683ee4be6d28adb49b94ea1335750e87d&tochange=cd84db0152b02a85e005058d19fe02a25cb1b012 Regression from bug 1179909.
Blocks: 1179909
status-firefox42:
--- → unaffected
status-firefox43:
--- → affected
status-firefox44:
--- → affected
Keywords: regressionwindow-wanted
Updated•9 years ago
|
Whiteboard: [MemShrink] → [MemShrink:P2]
Updated•9 years ago
|
Keywords: regression
Comment 4•8 years ago
|
||
This testcase no longer leaks for me in debug builds FWIW.
Comment 5•8 years ago
|
||
Jesse -- Can you retest this? Cpearce says he can't repro (see Comment 4).
Flags: needinfo?(jruderman)
Comment 6•7 years ago
|
||
Mass change P3->P4 to align with new Mozilla triage process.
Priority: P3 → P4
Comment 7•5 years ago
|
||
Needinfo self to try to reproduce, and close if necessary.
Flags: needinfo?(jruderman) → needinfo?(docfaraday)
Comment 8•5 years ago
|
||
This works well, now.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Flags: needinfo?(docfaraday)
Resolution: FIXED → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•