Closed
Bug 1243369
Opened 8 years ago
Closed 8 years ago
Intermittent leakcheck | tab process: 4712 bytes leaked (nsRunnable, nsThreadShutdownAckEvent)
Categories
(Core :: Graphics: Layers, defect)
Core
Graphics: Layers
Tracking
()
RESOLVED
DUPLICATE
of bug 1204784
Tracking | Status | |
---|---|---|
firefox47 | --- | affected |
People
(Reporter: jesup, Unassigned)
References
Details
(Keywords: intermittent-failure, memory-leak, Whiteboard: [gfx-noted])
https://treeherder.mozilla.org/logviewer.html#?job_id=20569959&repo=mozilla-inbound CompositorChild seems to have not shut down Perhaps it should use the AsyncShutdown blocking now used by MediaStreamGraph (bug 1239873)
Comment 1•8 years ago
|
||
There's some work in bug 1215265 around CompositorChild's shutdown.
Depends on: 1215265
Whiteboard: [gfx-noted]
Updated•8 years ago
|
Keywords: intermittent-failure,
mlk
FWIW this is the normal child side leak from gfx, which is currently suppressed, plus bug 1204784, which is what makes it show up.
Comment hidden (Intermittent Failures Robot) |
Flags: needinfo?(khuey)
Haven't seen this since on central since bug 1204784 landed.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(khuey)
Resolution: --- → DUPLICATE
Comment 5•8 years ago
|
||
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #4) > Haven't seen this since on central since bug 1204784 landed. > > *** This bug has been marked as a duplicate of bug 1204784 *** seems this is now permanent on aurora ? Do we need a fix there too ?
Flags: needinfo?(khuey)
Comment 6•8 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=mozilla-aurora&revision=48111b5e5b2b is the push where this seems to have started
Uplifting Bug 1204784 to Aurora (it already has approvals) should fix it ;)
Flags: needinfo?(khuey)
Comment 8•8 years ago
|
||
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #7) > Uplifting Bug 1204784 to Aurora (it already has approvals) should fix it ;) hm in bug 1204784 you said: "Note that this landed in time to make it to Aurora, so we only need it on Beta." but yeah will try to uplift that also to aurora :)
There were two separate changes. One needs to land on beta and aurora, the other needs to land on beta only.
Comment 10•8 years ago
|
||
ah ok thanks!
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•