[GTK] Require GTK+ >= 2.10

• Component: WebKitGTK
• Status: RESOLVED FIXED

[Gustavo Noronha (kov)]

As discussed in https://bugs.webkit.org/show_bug.cgi?id=22898, I believe it's about time we stop trying to support GTK+ versions bellow 2.10. The amount of functionality we lose, and the fact that we already depend on a very new libsoup version make it very difficult to argue for going extra miles to provide support for GTK+ version 2.8.

[Gustavo Noronha (kov)]

Created attachment 29128 [details]
require gtk+ >= 2.10


        Reviewed by NOBODY (OOPS!).

        Require GTK+ >= 2.10; 2.8 is already very old, and some very
        useful APIs are only available since 2.10.

        * configure.ac:

WebCore/

2009-03-31  Gustavo Noronha Silva  <[e-mail]>

        Reviewed by NOBODY (OOPS!).

        Remove pre-processor checks for GTK+ >= 2.10, since we now require
        at least that version.

        * platform/graphics/gtk/FontPlatformDataGtk.cpp:
        (WebCore::FontPlatformData::FontPlatformData):
        * platform/gtk/ContextMenuItemGtk.cpp:
        (WebCore::gtkStockIDFromContextMenuAction):
        * platform/gtk/KeyEventGtk.cpp:
        (WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
        * platform/gtk/LocalizedStringsGtk.cpp:
        (WebCore::contextMenuItemTagSelectAll):
        * platform/gtk/MouseEventGtk.cpp:
        (WebCore::PlatformMouseEvent::PlatformMouseEvent):
        * platform/gtk/PasteboardGtk.cpp:
        (WebCore::Pasteboard::writeSelection):
        (WebCore::Pasteboard::documentFragment):
        * platform/gtk/WheelEventGtk.cpp:
        (WebCore::PlatformWheelEvent::PlatformWheelEvent):

WebKit/gtk

2009-03-31  Gustavo Noronha Silva  <[e-mail]>

        Reviewed by NOBODY (OOPS!).

        Remove pre-processor checks for GTK+ >= 2.10, since we now require
        at least that version.

        * WebCoreSupport/EditorClientGtk.cpp:
        (WebKit::clipboard_clear_contents_cb):
        (WebKit::EditorClient::respondToChangedSelection):
        * webkit/webkitwebframe.cpp:
        * webkit/webkitwebview.cpp:
---
 ChangeLog                                          |    9 +++++++
 WebCore/ChangeLog                                  |   23 ++++++++++++++++++++
 .../platform/graphics/gtk/FontPlatformDataGtk.cpp  |    6 +---
 WebCore/platform/gtk/ContextMenuItemGtk.cpp        |    2 -
 WebCore/platform/gtk/KeyEventGtk.cpp               |    8 -------
 WebCore/platform/gtk/LocalizedStringsGtk.cpp       |    4 ---
 WebCore/platform/gtk/MouseEventGtk.cpp             |    8 -------
 WebCore/platform/gtk/PasteboardGtk.cpp             |    8 -------
 WebCore/platform/gtk/WheelEventGtk.cpp             |    8 -------
 WebKit/gtk/ChangeLog                               |   13 +++++++++++
 WebKit/gtk/WebCoreSupport/EditorClientGtk.cpp      |    4 ---
 WebKit/gtk/webkit/webkitwebframe.cpp               |   11 ---------
 WebKit/gtk/webkit/webkitwebview.cpp                |   10 --------
 configure.ac                                       |    2 +-
 14 files changed, 48 insertions(+), 68 deletions(-)

[Gustavo Noronha (kov)]

Comment on attachment 29128 [details]
require gtk+ >= 2.10

I am asking zecke to review this, specifically, as he is also the one who will need to make sure the bot has the appropriate version.

[Holger Freyther]

Comment on attachment 29128 [details]
require gtk+ >= 2.10

The buildbot has a jhbuild setup so Gtk+ and Glib should be fairly recent.

[Gustavo Noronha (kov)]

Landed as r42199.