[Webkit-unassigned] [Bug 33486] Upgrade 1.1.17->1.1.18 fails: GTK_WIDGET_TOPLEVEL' was not declared in this scope

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=33486


Xan Lopez <xan.lopez at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #46397|review+                     |review-
               Flag|                            |




--- Comment #3 from Xan Lopez <xan.lopez at gmail.com>  2010-01-14 05:04:07 PST ---
(From update of attachment 46397)
I'm going to comment about some particular issues in your code, but the overall
patch is wrong. You need to check the GTK+ version with GTK_CHECK_VERSION, and
only use your code if we are using 2.19.3 or newer. If you grep for that macro
you'll see examples of its usage. If you don't do this WebKitGTK+ wouldn't
compile with older GTK+ versions, and we don't want that.

>@@ -45,6 +45,7 @@
> #include <glib.h>
> #include <glib/gi18n-lib.h>
> #include <gtk/gtk.h>
>+#include <gtk/gtkwidget.h>

This is wrong, only the toplevel header should be included (I'm surprised this
works at all, I think GTK+ should complain).

> bool ChromeClient::canTakeFocus(FocusDirection)
> {
>-    return GTK_WIDGET_CAN_FOCUS(m_webView);
>+  return gtk_widget_get_can_focus((GtkWidget*)m_webView);

You can use GTK_WIDGET here, since it's safer and C-style casts are against the
style guidelines.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.