aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--ChangeLog6
-rw-r--r--embed/mozilla/GlobalHistory.cpp11
2 files changed, 8 insertions, 9 deletions
diff --git a/ChangeLog b/ChangeLog
index d1be0e315..2b3b6a635 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2005-08-22 Christian Persch <chpe@cvs.gnome.org>
+
+ * embed/mozilla/GlobalHistory.cpp:
+
+ Remove check for redirected URL. Fixes bug #313756.
+
2005-08-21 Christian Persch <chpe@cvs.gnome.org>
* src/ephy-tab.c: (ephy_tab_update_file_monitor):
diff --git a/embed/mozilla/GlobalHistory.cpp b/embed/mozilla/GlobalHistory.cpp
index ba6001b9a..6243fb0d2 100644
--- a/embed/mozilla/GlobalHistory.cpp
+++ b/embed/mozilla/GlobalHistory.cpp
@@ -58,11 +58,6 @@ NS_IMETHODIMP MozGlobalHistory::AddURI(nsIURI *aURI, PRBool aRedirect, PRBool aT
NS_ENSURE_ARG (aURI);
- if (aRedirect || !aToplevel)
- {
- return NS_OK;
- }
-
// filter out unwanted URIs such as chrome: etc
// The model is really if we don't know differently then add which basically
// means we are suppose to try all the things we know not to allow in and
@@ -70,9 +65,7 @@ NS_IMETHODIMP MozGlobalHistory::AddURI(nsIURI *aURI, PRBool aRedirect, PRBool aT
// against the most common case we know to allow in and go on and say yes
// to it.
- PRBool isHTTP = PR_FALSE;
- PRBool isHTTPS = PR_FALSE;
-
+ PRBool isHTTP = PR_FALSE, isHTTPS = PR_FALSE;
rv = aURI->SchemeIs("http", &isHTTP);
rv |= aURI->SchemeIs("https", &isHTTPS);
NS_ENSURE_SUCCESS (rv, NS_ERROR_FAILURE);
@@ -96,7 +89,7 @@ NS_IMETHODIMP MozGlobalHistory::AddURI(nsIURI *aURI, PRBool aRedirect, PRBool aT
nsEmbedCString spec;
rv = aURI->GetSpec(spec);
- NS_ENSURE_SUCCESS(rv, rv);
+ NS_ENSURE_TRUE (NS_SUCCEEDED(rv) && spec.Length(), rv);
ephy_history_add_page (mGlobalHistory, spec.get());