1153563 - ~CacheFile() releases main thread objects off the main thread

Status: RESOLVED INCOMPLETE

(Core :: Networking: Cache, defect)

Description

[Andrew McCreight [:mccr8]]

I found this crash stack:
https://crash-stats.mozilla.com/report/index/59cdac00-5ddc-47e0-bf0b-b65732150406

Maybe something is left in mObjsToRelease?

Comment 1

[Wayne Mery (:wsmwk)]

(In reply to Andrew McCreight [:mccr8] from comment #0)
> I found this crash stack:
> https://crash-stats.mozilla.com/report/index/59cdac00-5ddc-47e0-bf0b-
> b65732150406
> 
> Maybe something is left in mObjsToRelease?

 0 	xul.dll	NS_CycleCollectorSuspect3	xpcom/base/nsCycleCollector.cpp
1 	xul.dll	nsXMLElement::Release()	dom/xml/nsXMLElement.cpp
2 	xul.dll	nsRefPtr<mozilla::dom::RecordErrorEvent>::~nsRefPtr<mozilla::dom::RecordErrorEvent>()	xpcom/base/nsRefPtr.h
3 	xul.dll	mozilla::net::CacheFile::~CacheFile()	netwerk/cache2/CacheFile.cpp
4 	xul.dll	mozilla::net::CacheFile::`scalar deleting destructor'(unsigned int)	
5 	xul.dll	mozilla::net::CacheFile::Release()	netwerk/cache2/CacheFile.cpp
6 	xul.dll	nsRefPtr<mozilla::net::CacheFile>::~nsRefPtr<mozilla::net::CacheFile>()	xpcom/base/nsRefPtr.h
7 	xul.dll	mozilla::net::CacheEntry::~CacheEntry()	netwerk/cache2/CacheEntry.cpp
8 	xul.dll	mozilla::net::CacheEntry::`scalar deleting destructor'(unsigned int)	
9 	xul.dll	mozilla::net::CacheStorageService::MemoryPool::PurgeByFrecency(bool&, unsigned int)	netwerk/cache2/CacheStorageService.cpp

Comment 2

[Michal Novotny [:michal]]

The call stack doesn't make any sense. CacheFile doesn't hold reference to anything in dom.

Comment 3

[Andrew McCreight [:mccr8]]

I'll just mark this incomplete for now. I don't think I've seen this frequently.