From 9a618c1769f3542fd6dcb576fbb3c9007a06df3d Mon Sep 17 00:00:00 2001 From: "jst%mozilla.jstenback.com" Date: Wed, 29 Sep 2004 05:29:48 +0000 Subject: [PATCH] Fixing bug 261998. Don't fire the beforeunload event when loading a link to an anchor in the current document. r+sr=bzbarsky@mit.edu git-svn-id: svn://10.0.0.236/trunk@162934 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/docshell/base/nsDocShell.cpp | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/mozilla/docshell/base/nsDocShell.cpp b/mozilla/docshell/base/nsDocShell.cpp index 3e104693618..f4b00b47a4d 100644 --- a/mozilla/docshell/base/nsDocShell.cpp +++ b/mozilla/docshell/base/nsDocShell.cpp @@ -5228,19 +5228,6 @@ nsDocShell::InternalLoad(nsIURI * aURI, return rv; } - // Check if the page doesn't want to be unloaded. The javascript: - // protocol handler deals with this for javascript: URLs. - if (!bIsJavascript && mContentViewer) { - PRBool okToUnload; - rv = mContentViewer->PermitUnload(&okToUnload); - - if (NS_SUCCEEDED(rv) && !okToUnload) { - // The user chose not to unload the page, interrupt the - // load. - return NS_OK; - } - } - // // Load is being targetted at this docshell so return an error if the // docshell is in the process of being destroyed. @@ -5353,6 +5340,19 @@ nsDocShell::InternalLoad(nsIURI * aURI, } } + // Check if the page doesn't want to be unloaded. The javascript: + // protocol handler deals with this for javascript: URLs. + if (!bIsJavascript && mContentViewer) { + PRBool okToUnload; + rv = mContentViewer->PermitUnload(&okToUnload); + + if (NS_SUCCEEDED(rv) && !okToUnload) { + // The user chose not to unload the page, interrupt the + // load. + return NS_OK; + } + } + // Don't stop current network activity for javascript: URL's since // they might not result in any data, and thus nothing should be // stopped in those cases. In the case where they do result in