120772 2013-09-05 07:39:24 -0700 [GTK] [Meta] GtkActions and Stock Items are deprecated in gtk+ 3.10 2016-01-08 11:24:28 -0800 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED Gtk P2 Normal --- 120636 121686 1 spena webkit-unassigned berto brian.holt cgarcia eocanha mcatanzaro obzhirov zan oldest_to_newest 925024 0 spena 2013-09-05 07:39:24 -0700 WebKitGTK+ uses a number of symbols that have been deprecated in gtk+ 3.10: https://developer.gnome.org/gtk3/unstable/DeprecatedObjects.html, more specifically * GtkActions: $ fgrep -lr --include=*.{h,c,cpp} --exclude-dir=WebKitBuild GtkAction Source/WebKit2/UIProcess/API/gtk/WebKitContextMenuItem.cpp Source/WebKit2/UIProcess/API/gtk/tests/TestContextMenu.cpp Source/WebKit2/UIProcess/API/gtk/WebKitContextMenuItem.h Source/WebKit2/UIProcess/gtk/WebPopupMenuProxyGtk.h Source/WebKit2/UIProcess/gtk/WebPopupMenuProxyGtk.cpp Source/WebKit2/UIProcess/gtk/WebContextMenuProxyGtk.cpp Source/WTF/wtf/gobject/GTypedefs.h Source/WebKit/gtk/tests/testcontextmenu.c Source/WebKit/gtk/webkit/webkitglobals.cpp Source/WebCore/platform/graphics/gtk/FullscreenVideoControllerGtk.cpp Source/WebCore/platform/graphics/gtk/FullscreenVideoControllerGtk.h Source/WebCore/platform/ContextMenuItem.h Source/WebCore/platform/gtk/PopupMenuGtk.h Source/WebCore/platform/gtk/GtkPopupMenu.cpp Source/WebCore/platform/gtk/ContextMenuItemGtk.cpp Source/WebCore/platform/gtk/PopupMenuGtk.cpp Source/WebCore/platform/gtk/GtkPopupMenu.h Tools/MiniBrowser/gtk/BrowserWindow.c * Stock Items: $ fgrep -lir --include=*.{h,c,cpp} --exclude-dir=WebKitBuild gtk_stock Source/WebKit2/UIProcess/API/gtk/WebKitWebView.cpp Source/WebKit2/UIProcess/API/gtk/WebKitAuthenticationDialog.cpp Source/WebKit/gtk/webkit/webkitwebview.cpp Source/WebKit/gtk/webkit/webkitauthenticationdialog.cpp Source/WebCore/platform/graphics/gtk/ImageGtk.cpp Source/WebCore/platform/graphics/gtk/IconGtk.cpp Source/WebCore/platform/gtk/RenderThemeGtk.cpp Source/WebCore/platform/gtk/LocalizedStringsGtk.cpp Source/WebCore/platform/gtk/WebKitAuthenticationWidget.cpp Source/WebCore/platform/gtk/ContextMenuItemGtk.cpp Tools/MiniBrowser/gtk/BrowserDownloadsBar.c Tools/MiniBrowser/gtk/BrowserSettingsDialog.c Tools/MiniBrowser/gtk/BrowserWindow.c Tools/GtkLauncher/main.c (Some of the previous entries are false positives, since they are guarded by GTK_API_VERSION_2, so the proper way to do this would be to actually build against gtk+ 3.10 and check the error messages instead of simply grepping) 926629 1 berto 2013-09-09 03:06:37 -0700 I would be careful with removing deprecated symbols, I think it's important to check that the dependencies are not bumped because of this, else we would be making the webkitgtk build more difficult for no reason. 927666 2 zan 2013-09-11 00:15:59 -0700 This is at the moment breaking debug builds on the 2.2 stable branch, with GTK+ 3.10. As a workaround we could undefine the *_DISABLE_DEPRECATED macros in the case of debug builds. 927694 3 obzhirov 2013-09-11 02:30:36 -0700 I think you can already use GtkActions in current version of GTK. Not sure about stock items. 927695 4 obzhirov 2013-09-11 02:31:39 -0700 (In reply to comment #3) > I think you can already use GtkActions in current version of GTK. Not sure about stock items. Sorry I meant I can already remove GtkActions in current version of GTK. 927702 5 zan 2013-09-11 03:00:40 -0700 (In reply to comment #2) > This is at the moment breaking debug builds on the 2.2 stable branch, with GTK+ 3.10. As a workaround we could undefine the *_DISABLE_DEPRECATED macros in the case of debug builds. Those macro definitions were removed in r155509. https://trac.webkit.org/r155509 931363 6 cgarcia 2013-09-20 08:41:41 -0700 If the problem is that there are compile warnings, I wouldn't worry too much. We don't plan to bump GTK+ requirements, and I wouldn't spend time porting to new API either (I'm not even sure there's a replacement in current glib/GTK+ API for out uses cases), if the code is going to be full of #ifdefs. We expose GtkAction in the ContextMenu API and it works pretty well, so I would keep GtkAction and stock icons until we bump the requirements to GTK 4.0. 931373 7 obzhirov 2013-09-20 08:53:09 -0700 (In reply to comment #6) > If the problem is that there are compile warnings, I wouldn't worry too much. We don't plan to bump GTK+ requirements, and I wouldn't spend time porting to new API either (I'm not even sure there's a replacement in current glib/GTK+ API for out uses cases), if the code is going to be full of #ifdefs. We expose GtkAction in the ContextMenu API and it works pretty well, so I would keep GtkAction and stock icons until we bump the requirements to GTK 4.0. OK, so I guess it can be postponed for now. 932586 8 brian.holt 2013-09-24 09:02:34 -0700 (In reply to comment #6) > If the problem is that there are compile warnings, I wouldn't worry too much. We don't plan to bump GTK+ requirements, and I wouldn't spend time porting to new API either (I'm not even sure there's a replacement in current glib/GTK+ API for out uses cases), if the code is going to be full of #ifdefs. We expose GtkAction in the ContextMenu API and it works pretty well, so I would keep GtkAction and stock icons until we bump the requirements to GTK 4.0. Its not that there are just compile warning, building WebKit with debug symbols fails: $ ./autogen.sh --prefix=/opt/gnome3 --libdir=/opt/gnome3/lib64 --enable-debug Source/WebKit/gtk/webkit/webkitwebview.cpp:1309:53: error: 'GTK_STOCK_CANCEL' was not declared in this scope Source/WebKit/gtk/webkit/webkitwebview.cpp:1310:53: error: 'GTK_STOCK_OPEN' was not declared in this scope 932591 9 zan 2013-09-24 09:16:30 -0700 (In reply to comment #8) > (In reply to comment #6) > > If the problem is that there are compile warnings, I wouldn't worry too much. We don't plan to bump GTK+ requirements, and I wouldn't spend time porting to new API either (I'm not even sure there's a replacement in current glib/GTK+ API for out uses cases), if the code is going to be full of #ifdefs. We expose GtkAction in the ContextMenu API and it works pretty well, so I would keep GtkAction and stock icons until we bump the requirements to GTK 4.0. > > Its not that there are just compile warning, building WebKit with debug symbols fails: > $ ./autogen.sh --prefix=/opt/gnome3 --libdir=/opt/gnome3/lib64 --enable-debug > > Source/WebKit/gtk/webkit/webkitwebview.cpp:1309:53: error: 'GTK_STOCK_CANCEL' was not declared in this scope > Source/WebKit/gtk/webkit/webkitwebview.cpp:1310:53: error: 'GTK_STOCK_OPEN' was not declared in this scope I removed the disabling of deprecated API under debug builds in r155509. http://trac.webkit.org/changeset/155509 958263 10 cgarcia 2013-12-10 03:13:42 -0800 *** Bug 121686 has been marked as a duplicate of this bug. *** 958267 11 eocanha 2013-12-10 03:42:36 -0800 If migrating away from stock icons turns out to be a priority at some point, maybe https://bugs.webkit.org/attachment.cgi?id=218847 (from Bug 121686) might be considered. 1153904 12 mcatanzaro 2016-01-08 11:24:28 -0800 Stock item deprecations are trivial, patches welcome to get rid of those where they exist. No reason to keep a bug open for it without a patch, because they aren't hurting anything. GtkAction is part of our API, we cannot remove it. (In reply to comment #9) > I removed the disabling of deprecated API under debug builds in r155509. > http://trac.webkit.org/changeset/155509 This is all that matters.