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)
Core
IPC
Tracking
()
NEW
People
(Reporter: khuey, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: btpp-backlog)
Attachments
(1 file)
|
1.45 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
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.
| Reporter | ||
Comment 1•9 years ago
|
||
Attachment #8754025 -
Flags: review?(nfroyd)
Updated•9 years ago
|
Whiteboard: btpp-active
Comment 2•9 years ago
|
||
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+
| Reporter | ||
Comment 3•9 years ago
|
||
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
Updated•8 years ago
|
Priority: -- → P3
Whiteboard: btpp-active → btpp-backlog
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•