Fixes bug 71317. Leak increase on tinderbox. r=bryner sr=mscott,alecf

netwerk/base/src/nsSocketTransport.cpp

@@ -2227,6 +2227,7 @@ nsSocketRequest::OnStop()
             mStartFired = PR_TRUE;
         }
         mObserver->OnStopRequest(this, mContext, mStatus, nsnull);
+        mObserver = 0;
     }
     return NS_OK;
 }
@@ -2331,6 +2332,13 @@ nsSocketReadRequest::SetSocket(PRFileDesc *aSock)
     }
 }
 
+nsresult
+nsSocketReadRequest::OnStop()
+{
+    mListener = 0;
+    return nsSocketRequest::OnStop();
+}
+
 //
 // return:
 //   NS_OK                       to advance to next state
@@ -2439,6 +2447,13 @@ nsSocketWriteRequest::SetSocket(PRFileDesc *aSock)
     }
 }
 
+nsresult
+nsSocketWriteRequest::OnStop()
+{
+    mProvider = 0;
+    return nsSocketRequest::OnStop();
+}
+
 //
 // return:
 //   NS_OK                       to advance to next state

netwerk/base/src/nsSocketTransport.h

@@ -385,8 +385,8 @@ public:
 
     nsISupports *Context() { return mContext; }
     
-    nsresult OnStart();
-    nsresult OnStop();
+    virtual nsresult OnStart();
+    virtual nsresult OnStop();
 
 protected:
     nsSocketTransport          *mTransport;
@@ -410,6 +410,7 @@ public:
     void SetSocket(PRFileDesc *);
     void SetListener(nsIStreamListener *l) { mListener = l; }
 
+    nsresult OnStop();
     nsresult OnRead();
 
 private:
@@ -429,6 +430,7 @@ public:
     void SetSocket(PRFileDesc *);
     void SetProvider(nsIStreamProvider *p) { mProvider = p; }
 
+    nsresult OnStop();
     nsresult OnWrite();
 
 private: