Open Bug 975207 Opened 10 years ago Updated 2 years ago

HTTP cache v2: Let CacheStorageService keep a ref to CacheFileIOManager

Categories

(Core :: Networking: Cache, defect, P5)

Product:
Core
Component:
Networking: Cache
Type:
defect

Tracking

()

People

(Reporter: mayhemer, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-would-take]) 

As we know about the flaw with gInstance of CacheFileIOManager multi-thread access, I think it would be a good solution to just let the service (that is not released sooner then by xpcom shutdown since it's an xpcom service) reference the io manager.

This will prevent any (regardless how unlikely) shutdown crash.

Michal, what do you think?
Flags: needinfo?(michal.novotny)
Blocks: cache2afterenable
No longer blocks: cache2enable
Cleaning up the queue.  This is not that important.
Flags: needinfo?(michal.novotny)
Seems to be alright for now.
Whiteboard: [necko-would-take]
Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258
Priority: -- → P5
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.