summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMoonchild <moonchild@palemoon.org>2020-05-06 11:15:48 +0000
committerMoonchild <moonchild@palemoon.org>2020-05-06 11:15:48 +0000
commit3c38a37acdd8f30292d908577c0cef9740f2aee1 (patch)
tree291022c6d7176cf31b2f71e0d889f02d1db6748d
parentcc2f6543bd05160d1c36dbdacd4c6d143da767bc (diff)
downloadUXP-3c38a37acdd8f30292d908577c0cef9740f2aee1.tar
UXP-3c38a37acdd8f30292d908577c0cef9740f2aee1.tar.gz
UXP-3c38a37acdd8f30292d908577c0cef9740f2aee1.tar.lz
UXP-3c38a37acdd8f30292d908577c0cef9740f2aee1.tar.xz
UXP-3c38a37acdd8f30292d908577c0cef9740f2aee1.zip
[XPFE] Properly anchor XUL windows when tearing down
-rw-r--r--xpfe/appshell/nsXULWindow.cpp12
1 files changed, 2 insertions, 10 deletions
diff --git a/xpfe/appshell/nsXULWindow.cpp b/xpfe/appshell/nsXULWindow.cpp
index 8f2d9fde5..585085065 100644
--- a/xpfe/appshell/nsXULWindow.cpp
+++ b/xpfe/appshell/nsXULWindow.cpp
@@ -455,6 +455,8 @@ NS_IMETHODIMP nsXULWindow::Destroy()
if (mDestroying)
return NS_OK;
+ nsCOMPtr<nsIXULWindow> kungFuDeathGrip(this);
+
mozilla::AutoRestore<bool> guard(mDestroying);
mDestroying = true;
@@ -467,16 +469,6 @@ NS_IMETHODIMP nsXULWindow::Destroy()
if (parentWindow)
parentWindow->RemoveChildWindow(this);
- // let's make sure the window doesn't get deleted out from under us
- // while we are trying to close....this can happen if the docshell
- // we close ends up being the last owning reference to this xulwindow
-
- // XXXTAB This shouldn't be an issue anymore because the ownership model
- // only goes in one direction. When webshell container is fully removed
- // try removing this...
-
- nsCOMPtr<nsIXULWindow> placeHolder = this;
-
// Remove modality (if any) and hide while destroying. More than
// a convenience, the hide prevents user interaction with the partially
// destroyed window. This is especially necessary when the eldest window