Bug 566057: Remove obsolete managee tracking code in ContentProcessChild that's leading to use-after-free errors. r=jduell

dom/ipc/ContentProcessChild.cpp

@@ -61,7 +61,6 @@ namespace dom {
 ContentProcessChild* ContentProcessChild::sSingleton;
 
 ContentProcessChild::ContentProcessChild()
-    : mQuit(PR_FALSE)
 {
 }
 
@@ -86,37 +85,27 @@ PIFrameEmbeddingChild*
 ContentProcessChild::AllocPIFrameEmbedding()
 {
   nsRefPtr<TabChild> iframe = new TabChild();
-  NS_ENSURE_TRUE(iframe && NS_SUCCEEDED(iframe->Init()) &&
+  return NS_SUCCEEDED(iframe->Init()) ? iframe.forget().get() : NULL;
-                 mIFrames.AppendElement(iframe),
-                 nsnull);
-  return iframe.forget().get();
 }
 
 bool
 ContentProcessChild::DeallocPIFrameEmbedding(PIFrameEmbeddingChild* iframe)
 {
-    if (mIFrames.RemoveElement(iframe)) {
+    TabChild* child = static_cast<TabChild*>(iframe);
-      TabChild* child = static_cast<TabChild*>(iframe);
+    NS_RELEASE(child);
-      NS_RELEASE(child);
-    }
     return true;
 }
 
 PTestShellChild*
 ContentProcessChild::AllocPTestShell()
 {
-    PTestShellChild* testshell = new TestShellChild();
+    return new TestShellChild();
-    if (testshell && mTestShells.AppendElement(testshell)) {
-        return testshell;
-    }
-    delete testshell;
-    return nsnull;
 }
 
 bool
 ContentProcessChild::DeallocPTestShell(PTestShellChild* shell)
 {
-    mTestShells.RemoveElement(shell);
+    delete shell;
     return true;
 }
 
@@ -163,23 +152,12 @@ ContentProcessChild::RecvSetOffline(const PRBool& offline)
   return true;
 }
 
-void
-ContentProcessChild::Quit()
-{
-    NS_ASSERTION(mQuit, "Exiting uncleanly!");
-    mIFrames.Clear();
-    mTestShells.Clear();
-}
-
 void
 ContentProcessChild::ActorDestroy(ActorDestroyReason why)
 {
     if (AbnormalShutdown == why)
         NS_WARNING("shutting down because of crash!");
 
-    mQuit = PR_TRUE;
-    Quit();
-
     XRE_ShutdownChildProcess();
 }
 

dom/ipc/ContentProcessChild.h

@@ -89,15 +89,8 @@ private:
     NS_OVERRIDE
     virtual void ActorDestroy(ActorDestroyReason why);
 
-    void Quit();
-
     static ContentProcessChild* sSingleton;
 
-    nsTArray<PIFrameEmbeddingChild* > mIFrames;
-    nsTArray<nsAutoPtr<PTestShellChild> > mTestShells;
-
-    PRBool mQuit;
-
     DISALLOW_EVIL_CONSTRUCTORS(ContentProcessChild);
 };