Closed
Bug 669363
Opened 15 years ago
Closed 15 years ago
Pref service should assert/abort if used off the main thread
Categories
(Core :: Preferences: Backend, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 619487
People
(Reporter: jdm, Unassigned)
Details
Attachments
(1 file)
|
11.23 KB,
patch
|
Details | Diff | Splinter Review |
It's too easy to let non-main thread pref usage slip into the tree. We should shout really loudly if this happens, since it can cause crashes.
|
Reporter | |
Comment 1•15 years ago
|
||
|
||
Comment 2•15 years ago
|
||
I thought we had asserts for this at some point (and took them out?).
(In reply to comment #2)
> I thought we had asserts for this at some point (and took them out?).
Why would we remove them?
|
|
||
Comment 4•15 years ago
|
||
Because they were preventing daily use of the browser. Again, if I'm recalling correctly... maybe it wasn't the pref service.
|
|
Reporter | |
Comment 5•15 years ago
|
||
Oh yes, I now remember the whole hoopla in bug 580790.
|
|
Reporter | |
Updated•15 years ago
|
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•