From ad98d40a23784da3a62b3f8f9027e698626284e1 Mon Sep 17 00:00:00 2001
From: "cbiesinger%web.de" <cbiesinger%web.de>
Date: Wed, 27 Jul 2005 12:34:25 +0000
Subject: [PATCH] Bug 300863 Send onLocationChange even if we have no failed
 channel r=bz sr=darin a=bsmedberg

---
 docshell/base/nsDocShell.cpp | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

diff --git a/docshell/base/nsDocShell.cpp b/docshell/base/nsDocShell.cpp
index 9daece2771e..6f4b2f478b7 100644
--- a/docshell/base/nsDocShell.cpp
+++ b/docshell/base/nsDocShell.cpp
@@ -3011,7 +3011,10 @@ nsDocShell::LoadErrorPage(nsIURI *aURI, const PRUnichar *aURL,
         aURI->GetSpec(spec);
 
         nsCAutoString chanName;
-        aFailedChannel->GetName(chanName);
+        if (aFailedChannel)
+            aFailedChannel->GetName(chanName);
+        else
+            chanName.AssignLiteral("<no channel>");
 
         PR_LOG(gDocShellLog, PR_LOG_DEBUG,
                ("nsDocShell[%p]::LoadErrorPage(\"%s\", \"%s\", {...}, [%s])\n", this,
@@ -3022,6 +3025,9 @@ nsDocShell::LoadErrorPage(nsIURI *aURI, const PRUnichar *aURL,
     if (aFailedChannel) {
         mURIResultedInDocument = PR_TRUE;
         OnLoadingSite(aFailedChannel, PR_TRUE);
+    } else if (aURI) {
+        mURIResultedInDocument = PR_TRUE;
+        OnNewURI(aURI, nsnull, mLoadType, PR_TRUE);
     }
 
     nsCAutoString url;
@@ -6980,7 +6986,10 @@ nsDocShell::OnNewURI(nsIURI * aURI, nsIChannel * aChannel,
         aURI->GetSpec(spec);
 
         nsCAutoString chanName;
-        aChannel->GetName(chanName);
+        if (aChannel)
+            aChannel->GetName(chanName);
+        else
+            chanName.AssignLiteral("<no channel>");
 
         PR_LOG(gDocShellLog, PR_LOG_DEBUG,
                ("nsDocShell[%p]::OnNewURI(\"%s\", [%s], 0x%x)\n", this, spec.get(),
@@ -7032,6 +7041,9 @@ nsDocShell::OnNewURI(nsIURI * aURI, nsIChannel * aChannel,
     if (mCurrentURI)
         aURI->Equals(mCurrentURI, &equalUri);
 
+    PR_LOG(gDocShellLog, PR_LOG_DEBUG,
+           ("  shAvailable=%i updateHistory=%i equalURI=%i\n",
+            shAvailable, updateHistory, equalUri));
 
     /* If the url to be loaded is the same as the one already there,
      * and the original loadType is LOAD_NORMAL, LOAD_LINK, or
@@ -7185,7 +7197,10 @@ nsDocShell::AddToSessionHistory(nsIURI * aURI,
         aURI->GetSpec(spec);
 
         nsCAutoString chanName;
-        aChannel->GetName(chanName);
+        if (aChannel)
+            aChannel->GetName(chanName);
+        else
+            chanName.AssignLiteral("<no channel>");
 
         PR_LOG(gDocShellLog, PR_LOG_DEBUG,
                ("nsDocShell[%p]::AddToSessionHistory(\"%s\", [%s])\n", this, spec.get(),