Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

No need to enter the monitor in our destructor. Bug 371374 followup, r+sr=darin

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@221054 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
bzbarsky%mit.edu
2007-02-28 02:28:53 +00:00
parent 99d0de7701
commit 3c646ad803
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
mozilla/xpcom/threads/nsEventQueue.cpp
View File

@@ -58,14 +58,12 @@ nsEventQueue::nsEventQueue()
nsEventQueue::~nsEventQueue()
{
{
nsAutoMonitor mon(mMonitor);
// It'd be nice to be able to assert that no one else is holding the monitor,
// but NSPR doesn't really expose APIs for it.
NS_ASSERTION(IsEmpty(), "Non-empty event queue being destroyed; events being leaked.");
NS_ASSERTION(IsEmpty(), "Non-empty event queue being destroyed; events being leaked.");
if (mHead)
FreePage(mHead);
}
if (mHead)
FreePage(mHead);
if (mMonitor)
nsAutoMonitor::DestroyMonitor(mMonitor);