fix news not to open a new connection for ever post r=sspitzer, sr=mscott 65975

mailnews/news/src/nsNNTPProtocol.cpp

@@ -809,7 +809,11 @@ NS_IMETHODIMP nsNNTPProtocol::AsyncRead(nsIStreamListener *listener, nsISupports
           MarkCurrentMsgRead();
 
           if (NS_SUCCEEDED(rv)) // ONLY if we succeeded in actually starting the read should we return
+          {
+            m_ContentType = "";
+            m_channelListener = nsnull;
             return NS_OK;
+          }
         }
       }
     }
@@ -5444,6 +5448,7 @@ nsresult nsNNTPProtocol::CleanupAfterRunningUrl()
     m_channelContext = nsnull;
     m_channelListener = nsnull;
     m_loadGroup = nsnull;
+    mCallbacks = nsnull;
   }
   return NS_OK;
 }

mailnews/news/src/nsNntpService.cpp

@@ -839,11 +839,11 @@ nsNntpService::PostMessage(nsIFileSpec *fileToPost, const char *newsgroupsNames,
     mailnewsurl->RegisterListener(aUrlListener);
   
   // almost there...now create a nntp protocol instance to run the url in...
-  nsNNTPProtocol *nntpProtocol = nsnull;
+  nsCOMPtr <nsIURI> nntpURI = do_QueryInterface(nntpUrl);;
+  nsCOMPtr<nsINNTPProtocol> nntpProtocol ;
+  rv = GetProtocolForUri(nntpURI, aMsgWindow, getter_AddRefs(nntpProtocol));
 
-  // ### try to access cache - but can't find server yet...
-  nntpProtocol = new nsNNTPProtocol(mailnewsurl, aMsgWindow);
-  if (!nntpProtocol) return NS_ERROR_OUT_OF_MEMORY;;
+  if (!nntpProtocol) return NS_ERROR_OUT_OF_MEMORY;
   
   rv = nntpProtocol->Initialize(mailnewsurl, aMsgWindow);
   if (NS_FAILED(rv)) return rv;