Closed
Bug 1128486
Opened 9 years ago
Closed 9 years ago
Intermittent e10s leakcheck | tab process: 20056 bytes leaked (AsyncTransactionTrackersHolder, ChildDNSRecord, CipherSuiteChangeObserver, CompositorChild, CondVar, ...)
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla38
Tracking | Status | |
---|---|---|
e10s | + | --- |
firefox36 | --- | unaffected |
firefox37 | --- | unaffected |
firefox38 | --- | fixed |
firefox-esr31 | --- | unaffected |
People
(Reporter: cbook, Assigned: mccr8)
References
()
Details
(Keywords: intermittent-failure, memory-leak)
Attachments
(1 file)
1.28 KB,
patch
|
erahm
:
review+
|
Details | Diff | Splinter Review |
Ubuntu VM 12.04 x64 mozilla-inbound debug test mochitest-e10s-3 https://treeherder.mozilla.org/logviewer.html#?job_id=6155303&repo=mozilla-inbound 03:04:57 WARNING - TEST-UNEXPECTED-FAIL | leakcheck | tab process: 20056 bytes leaked (AsyncTransactionTrackersHolder, ChildDNSRecord, CipherSuiteChangeObserver, CompositorChild, CondVar, ...)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•9 years ago
|
Summary: Intermittent leakcheck | tab process: 20056 bytes leaked (AsyncTransactionTrackersHolder, ChildDNSRecord, CipherSuiteChangeObserver, CompositorChild, CondVar, ...) → Intermittent e10s leakcheck | tab process: 20056 bytes leaked (AsyncTransactionTrackersHolder, ChildDNSRecord, CipherSuiteChangeObserver, CompositorChild, CondVar, ...)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 5•9 years ago
|
||
The GeckoMediaPluginService leak slowly grows as people add more stuff. Until that's properly made multiprocess, we need to bump the threshold again.
Assignee: nobody → continuation
Component: General → Video/Audio
Assignee | ||
Comment 6•9 years ago
|
||
Attachment #8561488 -
Flags: review?(erahm)
Comment 7•9 years ago
|
||
Comment on attachment 8561488 [details] [diff] [review] Increase tab process leak threshold a little. Review of attachment 8561488 [details] [diff] [review]: ----------------------------------------------------------------- lgtm, presumably there's a bug for fixing this in GMP right?
Attachment #8561488 -
Flags: review?(erahm) → review+
Assignee | ||
Comment 8•9 years ago
|
||
(In reply to Eric Rahm [:erahm] from comment #7) > lgtm, presumably there's a bug for fixing this in GMP right? Yeah, sorry, I should have linked it here. Bug 1057908 is the bug. The GMP service depends on getting some profile shutdown message to do its teardown, but that isn't sent in the child process, so it just leaks. When it is run only in the parent process, this shouldn't be a problem any more.
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 10•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/55c2341c5520
Keywords: leave-open
Updated•9 years ago
|
tracking-e10s:
--- → +
Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
status-firefox36:
--- → unaffected
status-firefox37:
--- → unaffected
status-firefox38:
--- → fixed
status-firefox-esr31:
--- → unaffected
Keywords: leave-open
Target Milestone: --- → mozilla38
You need to log in
before you can comment on or make changes to this bug.
Description
•