Open Bug 1274028 Opened 9 years ago Updated 3 years ago

Shut down the IPC thread before the main thread

Categories

(Core :: IPC, defect, P3)

defect

Tracking

()

People

(Reporter: khuey, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: btpp-backlog)

Attachments

(1 file)

This doesn't actually work yet, because there are some things that try to do IPDL from dtors, so we'll need a few other fixes.
Whiteboard: btpp-active
Comment on attachment 8754025 [details] [diff] [review] Patch Review of attachment 8754025 [details] [diff] [review]: ----------------------------------------------------------------- OK, so we have no more threads. Anybody who's still doing chromium threads at this point gets what they deserve. Layers IPC got shut down earlier, so we should be good there. Things like AsyncShutdown and ProcessHangMonitor listen for things that have already happened at this point, so no danger of crashes from them. Browsing through all the things that happen between the two points you shuffled around, I don't see anything that should be doing IPC Let's try it. What do you need this for anyway? A little bit of explanation would be nice.
Attachment #8754025 - Flags: review?(nfroyd) → review+
So, the goal here was to turn the IPC thread into an nsThread. Which would require it to shut down when the thread manager shuts down. I ran out of time though :/
Assignee: khuey → nobody
Priority: -- → P3
Whiteboard: btpp-active → btpp-backlog
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: