From c35492e60895693bc5533563e3906847307a4ee9 Mon Sep 17 00:00:00 2001
From: "darin%meer.net" <darin%meer.net@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Wed, 8 Oct 2003 03:38:46 +0000
Subject: [PATCH] removing nsAutoLock workaround for bug 221331.

git-svn-id: svn://10.0.0.236/trunk@147732 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/netwerk/base/src/nsIOThreadPool.cpp | 67 ++++++++++-----------
 1 file changed, 32 insertions(+), 35 deletions(-)

diff --git a/mozilla/netwerk/base/src/nsIOThreadPool.cpp b/mozilla/netwerk/base/src/nsIOThreadPool.cpp
index 5717a1ad95a..c12cbb0835c 100644
--- a/mozilla/netwerk/base/src/nsIOThreadPool.cpp
+++ b/mozilla/netwerk/base/src/nsIOThreadPool.cpp
@@ -234,46 +234,43 @@ nsIOThreadPool::ThreadFunc(void *arg)
 
     LOG(("entering ThreadFunc\n"));
 
-    // XXX not using a nsAutoLock here because we need to temporarily unlock
-    // and relock in the inner loop.  nsAutoLock has a bug, causing it to warn
-    // of a bogus deadlock, which makes us avoid it here.  (see bug 221331)
-    PR_Lock(pool->mLock);
+    {
+        nsAutoLock lock(pool->mLock);
 
-    for (;;) {
-        // never wait if we are shutting down; always process queued events...
-        if (PR_CLIST_IS_EMPTY(&pool->mEventQ) && !pool->mShutdown) {
-            pool->mNumIdleThreads++;
-            PR_WaitCondVar(pool->mCV, THREAD_IDLE_TIMEOUT);
-            pool->mNumIdleThreads--;
+        for (;;) {
+            // never wait if we are shutting down; always process queued events...
+            if (PR_CLIST_IS_EMPTY(&pool->mEventQ) && !pool->mShutdown) {
+                pool->mNumIdleThreads++;
+                PR_WaitCondVar(pool->mCV, THREAD_IDLE_TIMEOUT);
+                pool->mNumIdleThreads--;
+            }
+
+            // if the queue is still empty, then kill this thread...
+            if (PR_CLIST_IS_EMPTY(&pool->mEventQ))
+                break;
+
+            // handle one event at a time: we don't want this one thread to hog
+            // all the events while other threads may be able to help out ;-)
+            do {
+                PLEvent *event = PLEVENT_FROM_LINK(PR_LIST_HEAD(&pool->mEventQ));
+                PR_REMOVE_AND_INIT_LINK(&event->link);
+
+                LOG(("event:%p\n", event));
+
+                // release lock!
+                lock.unlock();
+                PL_HandleEvent(event);
+                lock.lock();
+            }
+            while (!PR_CLIST_IS_EMPTY(&pool->mEventQ));
         }
 
-        // if the queue is still empty, then kill this thread...
-        if (PR_CLIST_IS_EMPTY(&pool->mEventQ))
-            break;
-
-        // handle one event at a time: we don't want this one thread to hog
-        // all the events while other threads may be able to help out ;-)
-        do {
-            PLEvent *event = PLEVENT_FROM_LINK(PR_LIST_HEAD(&pool->mEventQ));
-            PR_REMOVE_AND_INIT_LINK(&event->link);
-
-            LOG(("event:%p\n", event));
-
-            // release lock!
-            PR_Unlock(pool->mLock);
-            PL_HandleEvent(event);
-            PR_Lock(pool->mLock);
-        }
-        while (!PR_CLIST_IS_EMPTY(&pool->mEventQ));
+        // thread is dying... cleanup mThreads, unless of course if we are
+        // shutting down, in which case Shutdown will clean up mThreads for us.
+        if (!pool->mShutdown)
+            pool->mThreads[pool->GetCurrentThreadIndex()] = nsnull;
     }
 
-    // thread is dying... cleanup mThreads, unless of course if we are
-    // shutting down, in which case Shutdown will clean up mThreads for us.
-    if (!pool->mShutdown)
-        pool->mThreads[pool->GetCurrentThreadIndex()] = nsnull;
-
-    PR_Unlock(pool->mLock);
-
     // release our reference to the pool
     NS_RELEASE(pool);