rpms/gimp/devel gimp-2.6.6-help-browser-webkit.patch, 1.1, 1.2 gimp.spec, 1.193, 1.194

[Nils Philippsen]

Author: nphilipp

Update of /cvs/pkgs/rpms/gimp/devel
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv28129

Modified Files:
	gimp-2.6.6-help-browser-webkit.patch gimp.spec 
Log Message:
really fix help browser crash with new WebKit versions (#508301)

gimp-2.6.6-help-browser-webkit.patch:

Index: gimp-2.6.6-help-browser-webkit.patch
===================================================================
RCS file: /cvs/pkgs/rpms/gimp/devel/gimp-2.6.6-help-browser-webkit.patch,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -p -r1.1 -r1.2
--- gimp-2.6.6-help-browser-webkit.patch	27 Jun 2009 15:55:59 -0000	1.1
+++ gimp-2.6.6-help-browser-webkit.patch	29 Jun 2009 11:38:39 -0000	1.2
@@ -1,11 +1,23 @@
-commit 1d38c1c2daabb7dac94a37fa87caa0f1bb4d95f9
+commit 76c11122dd72e2c47675da394dadcca53c9069dd
 Author: Nils Philippsen <nils at redhat.com>
-Date:   Sat Jun 27 17:04:48 2009 +0200
+Date:   Mon Jun 29 13:34:13 2009 +0200
 
     patch: help-browser-webkit
     
     Squashed commit of the following:
     
+    commit 7f26593f67532850a2c3e1141f5c7f5e667b9064
+    Author: Sven Neumann <sven at gimp.org>
+    Date:   Tue May 12 09:39:21 2009 +0200
+    
+        help-browser: fix history menus
+    
+        The history menus located at the back and forward buttons stopped working
+        after a webkit update some time ago. With these changes the menus work
+        again, at least with recent webkit releases.
+    
+        Signed-off-by: Nils Philippsen <nils at redhat.com>
+    
     commit 7647788dbf37369b602361a6abd598388a4cb923
     Author: Sven Neumann <sven at gimp.org>
     Date:   Sat Mar 28 16:42:50 2009 +0000
@@ -23,6 +35,100 @@ Date:   Sat Jun 27 17:04:48 2009 +0200
     
         Signed-off-by: Nils Philippsen <nils at redhat.com>
 
+diff --git a/plug-ins/help-browser/dialog.c b/plug-ins/help-browser/dialog.c
+index 5ad6a06..159c67a 100644
+--- a/plug-ins/help-browser/dialog.c
++++ b/plug-ins/help-browser/dialog.c
+@@ -886,27 +886,30 @@ close_callback (GtkAction *action,
+ }
+ 
+ static void
+-menu_callback (GtkWidget            *menu,
+-               WebKitWebHistoryItem *item)
++menu_callback (GtkWidget *menu,
++               gpointer   data)
+ {
+-  browser_dialog_load (webkit_web_history_item_get_uri (item));
++  gint steps = GPOINTER_TO_INT (data);
++
++  webkit_web_view_go_back_or_forward (WEBKIT_WEB_VIEW (view), steps);
+ }
+ 
+-/*  this function unrefs the items and frees the list  */
+ static GtkWidget *
+-build_menu (GList *items)
++build_menu (const GList *items,
++            gboolean     back)
+ {
+-  GtkWidget *menu;
+-  GList     *list;
++  GtkWidget   *menu;
++  const GList *iter;
++  gint         steps;
+ 
+   if (! items)
+     return NULL;
+ 
+   menu = gtk_menu_new ();
+ 
+-  for (list = items; list; list = g_list_next (list))
++  for (iter = items, steps = 1; iter; iter = g_list_next (iter), steps++)
+     {
+-      WebKitWebHistoryItem *item = list->data;
++      WebKitWebHistoryItem *item = iter->data;
+       const gchar          *title;
+ 
+       title = webkit_web_history_item_get_title (item);
+@@ -918,16 +921,12 @@ build_menu (GList *items)
+           gtk_menu_shell_append (GTK_MENU_SHELL (menu), menu_item);
+           gtk_widget_show (menu_item);
+ 
+-          g_signal_connect_object (menu_item, "activate",
+-                                   G_CALLBACK (menu_callback),
+-                                   item, 0);
+-
+-          g_object_unref (item);
++          g_signal_connect (menu_item, "activate",
++                            G_CALLBACK (menu_callback),
++                            GINT_TO_POINTER (back ? - steps : steps));
+         }
+     }
+ 
+-  g_list_free (items);
+-
+   return menu;
+ }
+ 
+@@ -950,12 +949,12 @@ update_actions (void)
+ 
+   if (back_forward_list)
+     {
+-      GList *list;
++      const GList *list;
+ 
+       list = webkit_web_back_forward_list_get_back_list_with_limit (back_forward_list,
+                                                                     12);
+       gtk_menu_tool_button_set_menu (GTK_MENU_TOOL_BUTTON (button_prev),
+-                                     build_menu (list));
++                                     build_menu (list, TRUE));
+     }
+   else
+     {
+@@ -971,12 +970,12 @@ update_actions (void)
+ 
+   if (back_forward_list)
+     {
+-      GList *list;
++      const GList *list;
+ 
+       list = webkit_web_back_forward_list_get_forward_list_with_limit (back_forward_list,
+                                                                        12);
+       gtk_menu_tool_button_set_menu (GTK_MENU_TOOL_BUTTON (button_next),
+-                                     build_menu (list));
++                                     build_menu (list, FALSE));
+     }
+   else
+     {
 diff --git a/plug-ins/help-browser/help-browser.c b/plug-ins/help-browser/help-browser.c
 index efbabfe..d632d9c 100644
 --- a/plug-ins/help-browser/help-browser.c


Index: gimp.spec
===================================================================
RCS file: /cvs/pkgs/rpms/gimp/devel/gimp.spec,v
retrieving revision 1.193
retrieving revision 1.194
diff -u -p -r1.193 -r1.194
--- gimp.spec	27 Jun 2009 15:55:59 -0000	1.193
+++ gimp.spec	29 Jun 2009 11:38:40 -0000	1.194
@@ -31,7 +31,7 @@ Summary:        GNU Image Manipulation P
 Name:           gimp
 Epoch:          2
 Version:        2.6.6
-Release:        5%{?dist}
+Release:        6%{?dist}
 %define binver 2.6
 %define gimp_lang_ver 20
 %define interfacever 2.0
@@ -498,6 +498,9 @@ fi
 %{_libdir}/gimp/%{interfacever}/plug-ins/help-browser
 
 %changelog
+* Mon Jun 29 2009 Nils Philippsen <nils at redhat.com> - 2:2.6.6-6
+- really fix help browser crash with new WebKit versions (#508301)
+
 * Sat Jun 27 2009 Nils Philippsen <nils at redhat.com> - 2:2.6.6-5
 - fix help browser crash with new WebKit versions (#508301)
 - BR: webkitgtk-devel from F-11 on