fix hang in imap retry url, r=neil,a=Standard8 and for CLOSED TREE, bug 733731 RELEASE_11
authorDavid Bienvenu <bienvenu@nventure.com>
Tue, 13 Mar 2012 07:54:38 -0700
changeset 30765 832c448e5d0a30eddd3e2dbdaf8b7c72f43dc510
parent 30764 fed9432136dcc028bea812e532751240f5e0e200
child 30766 20e6ad6d901eaa2d780ef2cb3ee95b341f411cc4
child 30769 5e04902a70fa51a6361fa5c3bfdb4a450c2d2a5a
push id1
push userclokep@gmail.com
push dateMon, 07 May 2018 22:45:56 +0000
treeherdercomm-esr60@57eacde5ef40 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersneil, Standard8
bugs733731
fix hang in imap retry url, r=neil,a=Standard8 and for CLOSED TREE, bug 733731
mailnews/imap/src/nsImapProtocol.cpp
--- a/mailnews/imap/src/nsImapProtocol.cpp
+++ b/mailnews/imap/src/nsImapProtocol.cpp
@@ -1863,32 +1863,30 @@ bool nsImapProtocol::ProcessCurrentURL()
         m_useIdle = PR_FALSE;
     }
   }
   return anotherUrlRun;
 }
 
 bool nsImapProtocol::RetryUrl()
 {
-  PR_CEnterMonitor(this);
   nsCOMPtr <nsIImapUrl> kungFuGripImapUrl = m_runningUrl;
   nsCOMPtr <nsIImapMockChannel> saveMockChannel;
 
   // the mock channel might be null - that's OK.
   if (m_imapServerSink)
     (void) m_imapServerSink->PrepareToRetryUrl(kungFuGripImapUrl, getter_AddRefs(saveMockChannel));
 
   ReleaseUrlState(PR_TRUE);
   nsresult rv;
   nsCOMPtr<nsIImapIncomingServer> imapServer  = do_QueryReferent(m_server, &rv);
   if (NS_SUCCEEDED(rv))
     imapServer->RemoveConnection(this);
   if (m_imapServerSink)
     m_imapServerSink->RetryUrl(kungFuGripImapUrl, saveMockChannel);
-  PR_CExitMonitor(this);
   return (m_imapServerSink != nsnull); // we're running a url (the same url)
 }
 
 // ignoreBadAndNOResponses --> don't throw a error dialog if this command results in a NO or Bad response
 // from the server..in other words the command is "exploratory" and we don't really care if it succeeds or fails.
 void nsImapProtocol::ParseIMAPandCheckForNewMail(const char* commandString, bool aIgnoreBadAndNOResponses)
 {
     if (commandString)