Bug 482939 - Need to use nsAutoMonitor::New/DestroyMonitor when using nsAutoMonitor to grab monitors; r=kaie a=bsmedberg

This commit is contained in:
Robert O'Callahan 2010-11-26 12:19:32 -05:00
parent 3aad557531
commit cb8fee773e
3 changed files with 7 additions and 6 deletions

View File

@ -173,7 +173,7 @@ nsSecureBrowserUIImpl::nsSecureBrowserUIImpl()
, mOnStateLocationChangeReentranceDetection(0)
#endif
{
mMonitor = PR_NewMonitor();
mMonitor = nsAutoMonitor::NewMonitor("security.secureBrowserUIImplMonitor");
mTransferringRequests.ops = nsnull;
ResetStateTracking();
@ -190,7 +190,7 @@ nsSecureBrowserUIImpl::~nsSecureBrowserUIImpl()
mTransferringRequests.ops = nsnull;
}
if (mMonitor)
PR_DestroyMonitor(mMonitor);
nsAutoMonitor::DestroyMonitor(mMonitor);
}
NS_IMPL_THREADSAFE_ISUPPORTS6(nsSecureBrowserUIImpl,

View File

@ -120,13 +120,13 @@ NS_IMPL_THREADSAFE_ISUPPORTS3(nsCertOverrideService,
nsCertOverrideService::nsCertOverrideService()
{
monitor = PR_NewMonitor();
monitor = nsAutoMonitor::NewMonitor("security.certOverrideServiceMonitor");
}
nsCertOverrideService::~nsCertOverrideService()
{
if (monitor)
PR_DestroyMonitor(monitor);
nsAutoMonitor::DestroyMonitor(monitor);
}
nsresult
@ -914,3 +914,4 @@ nsCertOverrideService::GetHostWithPort(const nsACString & aHostName, PRInt32 aPo
}
_retval.Assign(hostPort);
}

View File

@ -62,13 +62,13 @@ NS_IMPL_THREADSAFE_ISUPPORTS1(nsRecentBadCertsService,
nsRecentBadCertsService::nsRecentBadCertsService()
:mNextStorePosition(0)
{
monitor = PR_NewMonitor();
monitor = nsAutoMonitor::NewMonitor("security.recentBadCertsMonitor");
}
nsRecentBadCertsService::~nsRecentBadCertsService()
{
if (monitor)
PR_DestroyMonitor(monitor);
nsAutoMonitor::DestroyMonitor(monitor);
}
nsresult