Closed
Bug 1067046
Opened 9 years ago
Closed 2 years ago
Don't try to shut down the PBackground thread until application shutdown
Categories
(Core :: DOM: Content Processes, defect)
Core
DOM: Content Processes
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: bent.mozilla, Assigned: bent.mozilla)
References
Details
Attachments
(1 file)
4.64 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
The PBackground thread stays alive as long as you have PBackground actors. With IndexedDB now using PBackground the main thread actor will stay alive until app shutdown, so there's no real point trying to reclaim the PBackground thread until app shutdown any more. This will also let us simplify PBackground actors a bit.
Attachment #8489045 -
Flags: review?(khuey)
Comment on attachment 8489045 [details] [diff] [review] Patch, v1 Review of attachment 8489045 [details] [diff] [review]: ----------------------------------------------------------------- Why do we need sShutdownTimer at all? Why not just create it when we need it?
Attachment #8489045 -
Flags: review?(khuey) → review+
Comment 2•2 years ago
|
||
The code there changed significantly in the meantime and there is no more ShutdownBackgroundThread
here. I assume we improved the situation in some other way.
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•