Closed Bug 814721 Opened 12 years ago Closed 12 years ago

Fix leaks from try run in bug 781534 for basic gum automation

Categories

(Core :: WebRTC: Audio/Video, defect, P1)

defect

Tracking

()

RESOLVED DUPLICATE of bug 802538

People

(Reporter: jsmith, Assigned: jesup)

Details

(Keywords: memory-leak, Whiteboard: [getUserMedia][blocking-gum+])

Attachments

(1 file)

The try run from https://tbpl.mozilla.org/?tree=Try&rev=f7e037dc5360 failed additionally due to leaks: TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 3705 bytes during test execution TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1 instance of CondVar with size 32 bytes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 2 instances of ImageContainer with size 128 bytes each (256 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 2 instances of ImageFactory with size 16 bytes each (32 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 2 instances of MediaEngineDefaultAudioSource with size 56 bytes each (112 bytes total) TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 2 instances of MediaEngineDefaultVideoSource with size 72 bytes each (144 bytes total) We need to fix these leaks to pref on the automation from bug 781534.
Blocks: 781534
Whiteboard: [getUserMedia][blocking-gum+][automation-blocked]
Keywords: mlk
Jason, can you please test locally which part of the test leaks? That would help us to nail that down and get solved faster.
Blocks: 812648
Blocks: 814807
No longer blocks: 781534
Attached patch leaking mochitest (deleted) — Splinter Review
The following basic gUM test already leaks a good amount of data. It's a bit different to what Jason gave in his initially comment, but lets get started with it because it could also fix the other leaks in-place. TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1483 bytes during test execution TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1 instance of CondVar with size 32 bytes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1 instance of MediaEngineDefaultAudioSource with size 56 bytes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1 instance of MediaStream with size 192 bytes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1 instance of MediaStreamGraph with size 16 bytes TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 1 instance of MediaStreamListener with size 16 bytes
No longer blocks: 814807
Assignee: nobody → rjesup
Priority: -- → P1
Blocks: 823520
No longer blocks: 812648
Depends on: 824362
Blocks: 824362
No longer depends on: 824362
Will be fixed by bug 802538. Duping on that.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
No longer blocks: 823520, 824362
Whiteboard: [getUserMedia][blocking-gum+][automation-blocked] → [getUserMedia][blocking-gum+]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: