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)

x86
Linux
defect
Not set
normal

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)

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, ...)
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, ...)
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
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+
(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.
This really does not need a try run.
Keywords: checkin-needed
Keywords: checkin-needed
tracking-e10s: --- → +
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Keywords: leave-open
Target Milestone: --- → mozilla38
Blocks: 1051230
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: