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)
Core
WebRTC: Audio/Video
Tracking
()
RESOLVED
DUPLICATE
of bug 802538
People
(Reporter: jsmith, Assigned: jesup)
Details
(Keywords: memory-leak, Whiteboard: [getUserMedia][blocking-gum+])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Updated•12 years ago
|
Blocks: 781534
Whiteboard: [getUserMedia][blocking-gum+][automation-blocked]
Comment 1•12 years ago
|
||
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
Comment 2•12 years ago
|
||
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
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → rjesup
Assignee | ||
Updated•12 years ago
|
Priority: -- → P1
Updated•12 years ago
|
Updated•12 years ago
|
Reporter | ||
Comment 3•12 years ago
|
||
Will be fixed by bug 802538. Duping on that.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
Reporter | ||
Updated•12 years ago
|
Updated•12 years ago
|
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.
Description
•