2008-08-01 Wouter Bolsterlee <uws@xs4all.nl>
authorjmalonzo@webkit.org <jmalonzo@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 2 Aug 2008 05:12:11 +0000 (05:12 +0000)
committerjmalonzo@webkit.org <jmalonzo@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 2 Aug 2008 05:12:11 +0000 (05:12 +0000)
        https://bugs.webkit.org/show_bug.cgi?id=20035
        [GTK] Fix return value for webkit_web_view_go_to_back_forward_item()

        * webkit/webkitwebview.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@35522 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/gtk/ChangeLog
WebKit/gtk/webkit/webkitwebview.cpp

index 32439e0..6326773 100644 (file)
@@ -1,3 +1,10 @@
+2008-08-01  Wouter Bolsterlee  <uws@xs4all.nl>
+
+        https://bugs.webkit.org/show_bug.cgi?id=20035
+        [GTK] Fix return value for webkit_web_view_go_to_back_forward_item()
+
+        * webkit/webkitwebview.cpp:
+
 2008-07-30  Marco Barisione  <marco.barisione@collabora.co.uk>
 
         Reviewed by Holger.
index 7f99949..e06dcbf 100644 (file)
@@ -1520,7 +1520,10 @@ gboolean webkit_web_view_go_to_back_forward_item(WebKitWebView* webView, WebKitW
     g_return_val_if_fail(WEBKIT_IS_WEB_VIEW(webView), FALSE);
     g_return_val_if_fail(WEBKIT_IS_WEB_HISTORY_ITEM(item), FALSE);
 
-    // FIXME: This always returns TRUE
+    WebKitWebBackForwardList* backForwardList = webkit_web_view_get_back_forward_list(webView);
+    if (!webkit_web_back_forward_list_contains_item(backForwardList, item))
+        return FALSE;
+
     core(webView)->goToItem(core(item), FrameLoadTypeIndexedBackForward);
     return TRUE;
 }