Bug 998377 - Stop calling ReleaseWrapper in DelayedDisconnect(). r=smaug
authorBobby Holley <bobbyholley@gmail.com>
Fri, 18 Apr 2014 16:00:06 -0700
changeset 179664 d8c4225f799fbe0a2ae450ecce328a87e95a5155
parent 179663 e5b5eed30c6ec8887c464db72ff1f01ca69d9efd
child 179665 1af6111edd497cd8f220a174ae0245379724641f
push id272
push userpvanderbeken@mozilla.com
push dateMon, 05 May 2014 16:31:18 +0000
reviewerssmaug
bugs998377
milestone31.0a1
Bug 998377 - Stop calling ReleaseWrapper in DelayedDisconnect(). r=smaug AFAICT, nobody ever calls PreserveWrapper. Presumably, the CC machinery should take care of us here.
content/base/src/nsInProcessTabChildGlobal.cpp
--- a/content/base/src/nsInProcessTabChildGlobal.cpp
+++ b/content/base/src/nsInProcessTabChildGlobal.cpp
@@ -231,17 +231,16 @@ nsInProcessTabChildGlobal::DelayedDiscon
     static_cast<nsFrameMessageManager*>(mMessageManager.get())->Disconnect();
     mMessageManager = nullptr;
   }
   if (mListenerManager) {
     mListenerManager->Disconnect();
   }
 
   if (!mLoadingScript) {
-    ReleaseWrapper(static_cast<EventTarget*>(this));
   } else {
     mDelayedDisconnect = true;
   }
 }
 
 NS_IMETHODIMP_(nsIContent *)
 nsInProcessTabChildGlobal::GetOwnerContent()
 {