Closed Bug 1117203 Opened 10 years ago Closed 9 years ago

ImageBridgeChild leaks in e10s content process

Categories

(Core :: Graphics: Layers, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1215265
Tracking Status
e10s + ---

People

(Reporter: mccr8, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [MemShrink:P3])

Attachments

(1 file, 1 obsolete file)

See bug 1065536 for context.
This was r+'d in bug 1065536, but it is causing Windows-only test failures: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=43a1611643d8
tracking-e10s: --- → ?
Whiteboard: [MemShrink:P2] → [MemShrink]
Whiteboard: [MemShrink] → [MemShrink:P3]
We marked this P3 because it seems like this isn't really a leak, but just some per-process singleton we're not tearing down properly in leak checking builds. The main goal here is to remove the noise from the leak logs so we can detect real leaks. Hopefully I'll get back to this at some point, but not in the near term.
Assignee: continuation → nobody
No longer depends on: 1065536
Blocks: 1213464
Blocks: 1122045
Attachment #8543383 - Attachment is obsolete: true
Comment on attachment 8675165 [details] [diff] [review] Shut down ImageBridgeChild in content processes. Nical, is there some reason we can't just call Shutdown in the child process, too? I don't know what all is going on in the code, but a try run was green, and it seems to fix the ImageBridgeChild leak. https://treeherder.mozilla.org/#/jobs?repo=try&revision=0f75253cd574
Attachment #8675165 - Flags: feedback?(nical.bugzilla)
Comment on attachment 8675165 [details] [diff] [review] Shut down ImageBridgeChild in content processes. Review of attachment 8675165 [details] [diff] [review]: ----------------------------------------------------------------- In short: I don't remember. The longer version is: I looked into hg logs and this comes from bug 774388 which attempted to fix a whole bunch of issues with the shutdown and turned into an epic saga. The reason is probably buried in there, I assume that we had to cut some corners to get the thing landed. I am keen on trying this, although you should watch out for intermittent issues after you land this because we had many of those in bug 774388 and it could be the reason ImageBridgeChild ended up leaked in content processes. You definitely want a try push with all platforms for this one!
Attachment #8675165 - Flags: feedback?(nical.bugzilla) → feedback+
No longer blocks: 1213464
roc is fixing this in bug 1215265.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: