Backing out previous checkin since leaks went up

content/base/src/nsDocument.cpp

@@ -5433,17 +5433,26 @@ nsDocument::Destroy()
   if (mIsGoingAway)
     return;
 
+  PRInt32 count = mChildren.ChildCount();
 
   mIsGoingAway = PR_TRUE;
-
-  PRUint32 i, count = mChildren.ChildCount();
-  for (i = 0; i < count; ++i) {
-    nsNodeUtils::DestroySubtree(mChildren.ChildAt(i));
+  DestroyLinkMap();
+  for (PRInt32 indx = 0; indx < count; ++indx) {
+    // XXXbz what we _should_ do here is to clear mChildren and null out
+    // mRootContent.  If we did this (or at least the latter), we could remove
+    // the silly null-checks in nsHTMLDocument::MatchLinks.  Unfortunately,
+    // doing that introduces several problems:
+    // 1) Focus issues (see bug 341730).  The fix for bug 303260 may fix these.
+    // 2) Crashes in OnPageHide if it fires after Destroy.  See bug 303260
+    //    comments 9 and 10.
+    // So we're just creating an inconsistent DOM for now and hoping.  :(
+    mChildren.ChildAt(indx)->UnbindFromTree();
   }
-
   mLayoutHistoryState = nsnull;
 
   nsContentList::OnDocumentDestroy(this);
+  delete mContentWrapperHash;
+  mContentWrapperHash = nsnull;
 }
 
 already_AddRefed<nsILayoutHistoryState>

content/base/src/nsGenericElement.cpp

@@ -3271,11 +3271,8 @@ NS_IMPL_CYCLE_COLLECTION_UNLINK_BEGIN(nsGenericElement)
   // Unlink any DOM slots of interest.
   {
     nsDOMSlots *slots = tmp->GetExistingDOMSlots();
-    if (slots) {
+    if (slots)
       slots->mAttributeMap = nsnull;
-      if (tmp->IsNodeOfType(nsINode::eXUL))
-        NS_IF_RELEASE(slots->mControllers);
-    }
   }
 NS_IMPL_CYCLE_COLLECTION_UNLINK_END
 

content/base/src/nsNodeUtils.cpp

@@ -677,27 +677,3 @@ nsNodeUtils::UnlinkUserData(nsINode *aNode)
                                                       DOM_USER_DATA_HANDLER);
   }
 }
-
-/* static */
-void
-nsNodeUtils::DestroySubtree(nsIContent* aRoot)
-{
-  nsXULElement* xul = nsXULElement::FromContent(aRoot);
-  if (xul) {
-    nsGenericElement::nsDOMSlots* slots = xul->GetExistingDOMSlots();
-    if (slots) {
-      NS_IF_RELEASE(slots->mControllers);
-    }
-  }
-
-  nsIDocument *document = aRoot->GetOwnerDoc();
-  if (document) {
-    document->BindingManager()->ChangeDocumentFor(aRoot, document, nsnull);
-    document->ClearBoxObjectFor(aRoot);
-  }
-
-  PRUint32 i, count = aRoot->GetChildCount();
-  for (i = 0; i < count; ++i) {
-    DestroySubtree(aRoot->GetChildAt(i));
-  }
-}

content/base/src/nsNodeUtils.h

@@ -258,14 +258,6 @@ public:
    */
   static void UnlinkUserData(nsINode *aNode);
 
-  /**
-   * Remove neccesary components of all nodes in a subtree to avoid leaking.
-   * So far this removes XBL bindings and XUL controllers.
-   *
-   * @param aRoot the node that is the root of the subtree to clear.
-   */
-  static void DestroySubtree(nsIContent* aRoot);
-
 private:
   friend PLDHashOperator PR_CALLBACK
     AdoptFunc(nsAttrHashKey::KeyType aKey, nsIDOMNode *aData, void* aUserArg);

content/html/content/src/nsHTMLFormElement.cpp

@@ -363,7 +363,7 @@ protected:
   /** The request currently being submitted */
   nsCOMPtr<nsIRequest> mSubmittingRequest;
   /** The web progress object we are currently listening to */
-  nsWeakPtr mWebProgress;
+  nsCOMPtr<nsIWebProgress> mWebProgress;
 
   /** The default submit element -- WEAK */
   nsIFormControl* mDefaultSubmitElement;
@@ -1057,12 +1057,10 @@ nsHTMLFormElement::SubmitSubmission(nsIFormSubmission* aFormSubmission)
     PRBool pending = PR_FALSE;
     mSubmittingRequest->IsPending(&pending);
     if (pending && !schemeIsJavaScript) {
-      nsCOMPtr<nsIWebProgress> webProgress = do_GetInterface(docShell);
-      NS_ASSERTION(webProgress, "nsIDocShell not converted to nsIWebProgress!");
-      rv = webProgress->AddProgressListener(this, nsIWebProgress::NOTIFY_STATE_ALL);
+      mWebProgress = do_GetInterface(docShell);
+      NS_ASSERTION(mWebProgress, "nsIDocShell not converted to nsIWebProgress!");
+      rv = mWebProgress->AddProgressListener(this, nsIWebProgress::NOTIFY_STATE_ALL);
       NS_ENSURE_SUBMIT_SUCCESS(rv);
-      mWebProgress = do_GetWeakReference(webProgress);
-      NS_ASSERTION(mWebProgress, "can't hold weak ref to webprogress!");
     } else {
       ForgetCurrentSubmission();
     }
@@ -1634,11 +1632,10 @@ nsHTMLFormElement::ForgetCurrentSubmission()
   mNotifiedObservers = PR_FALSE;
   mIsSubmitting = PR_FALSE;
   mSubmittingRequest = nsnull;
-  nsCOMPtr<nsIWebProgress> webProgress = do_QueryReferent(mWebProgress);
-  if (webProgress) {
-    webProgress->RemoveProgressListener(this);
+  if (mWebProgress) {
+    mWebProgress->RemoveProgressListener(this);
+    mWebProgress = nsnull;
   }
-  mWebProgress = nsnull;
 }
 
 // nsIWebProgressListener

content/xtf/src/nsXTFElementWrapper.cpp

@@ -104,15 +104,8 @@ nsXTFElementWrapper::Init()
 //----------------------------------------------------------------------
 // nsISupports implementation
 
-NS_IMPL_ADDREF_INHERITED(nsXTFElementWrapper, nsXTFElementWrapperBase)
-NS_IMPL_RELEASE_INHERITED(nsXTFElementWrapper, nsXTFElementWrapperBase)
-
-NS_IMPL_CYCLE_COLLECTION_CLASS(nsXTFElementWrapper)
-NS_IMPL_CYCLE_COLLECTION_TRAVERSE_BEGIN_INHERITED(nsXTFElementWrapper,
-                                                  nsXTFElementWrapperBase)
-  NS_IMPL_CYCLE_COLLECTION_TRAVERSE_NSCOMPTR(mXTFElement)
-  NS_IMPL_CYCLE_COLLECTION_TRAVERSE_NSCOMPTR(mAttributeHandler)
-NS_IMPL_CYCLE_COLLECTION_TRAVERSE_END
+NS_IMPL_ADDREF_INHERITED(nsXTFElementWrapper,nsXTFElementWrapperBase)
+NS_IMPL_RELEASE_INHERITED(nsXTFElementWrapper,nsXTFElementWrapperBase)
 
 NS_IMETHODIMP
 nsXTFElementWrapper::QueryInterface(REFNSIID aIID, void** aInstancePtr)

content/xtf/src/nsXTFElementWrapper.h

@@ -65,8 +65,6 @@ public:
 
   // nsISupports interface
   NS_DECL_ISUPPORTS_INHERITED
-  NS_DECL_CYCLE_COLLECTION_CLASS_INHERITED_NO_UNLINK(nsXTFElementWrapper,
-                                                     nsXTFElementWrapperBase)
 
   // nsIXTFElementWrapper
   NS_DECL_NSIXTFELEMENTWRAPPER