Lerking/gtk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Before delivering an event to a widget, check that (if relevant), the

Browse Source 
Fri Jun  6 11:07:33 2003  Owen Taylor  <otaylor@redhat.com>

        * gtk/gtkwidget.c (event_window_still_viewable):
        Before delivering an event to a widget, check that
        (if relevant), the event's window is still viewable.
        (#105642, Dennis Björklund)

        * gdk/gdkwindow.c (_gdk_window_destroy_hierarchy):
        NULL out private->parent, since after destruction
        it might not be valid any more.

        * gdk/gdkwindow.c (gdk_window_is_viewable): Fix some
        accesses before g_return_val_if_fail(). Treat
        DESTROYED windows as unmapped.
This commit is contained in:
Owen Taylor
2003-06-09 01:54:48 +00:00
committed by Owen Taylor
parent afa7fb3730
commit 995098e65e
6 changed files with 128 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
ChangeLog
View File

@@ -1,4 +1,19 @@
Sun Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
Fri Jun 6 11:07:33 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkwidget.c (event_window_still_viewable):
Before delivering an event to a widget, check that
(if relevant), the event's window is still viewable.
(#105642, Dennis Björklund)
* gdk/gdkwindow.c (_gdk_window_destroy_hierarchy):
NULL out private->parent, since after destruction
it might not be valid any more.
* gdk/gdkwindow.c (gdk_window_is_viewable): Fix some
accesses before g_return_val_if_fail(). Treat
DESTROYED windows as unmapped.
n Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
* gdk/gdkpixbuf-drawable.c (G8fromRGB565): swap G and B.
Fixes (#114669)

17
ChangeLog.pre-2-10
View File

@@ -1,4 +1,19 @@
Sun Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
Fri Jun 6 11:07:33 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkwidget.c (event_window_still_viewable):
Before delivering an event to a widget, check that
(if relevant), the event's window is still viewable.
(#105642, Dennis Björklund)
* gdk/gdkwindow.c (_gdk_window_destroy_hierarchy):
NULL out private->parent, since after destruction
it might not be valid any more.
* gdk/gdkwindow.c (gdk_window_is_viewable): Fix some
accesses before g_return_val_if_fail(). Treat
DESTROYED windows as unmapped.
n Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
* gdk/gdkpixbuf-drawable.c (G8fromRGB565): swap G and B.
Fixes (#114669)

17
ChangeLog.pre-2-4
View File

@@ -1,4 +1,19 @@
Sun Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
Fri Jun 6 11:07:33 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkwidget.c (event_window_still_viewable):
Before delivering an event to a widget, check that
(if relevant), the event's window is still viewable.
(#105642, Dennis Björklund)
* gdk/gdkwindow.c (_gdk_window_destroy_hierarchy):
NULL out private->parent, since after destruction
it might not be valid any more.
* gdk/gdkwindow.c (gdk_window_is_viewable): Fix some
accesses before g_return_val_if_fail(). Treat
DESTROYED windows as unmapped.
n Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
* gdk/gdkpixbuf-drawable.c (G8fromRGB565): swap G and B.
Fixes (#114669)

17
ChangeLog.pre-2-6
View File

@@ -1,4 +1,19 @@
Sun Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
Fri Jun 6 11:07:33 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkwidget.c (event_window_still_viewable):
Before delivering an event to a widget, check that
(if relevant), the event's window is still viewable.
(#105642, Dennis Björklund)
* gdk/gdkwindow.c (_gdk_window_destroy_hierarchy):
NULL out private->parent, since after destruction
it might not be valid any more.
* gdk/gdkwindow.c (gdk_window_is_viewable): Fix some
accesses before g_return_val_if_fail(). Treat
DESTROYED windows as unmapped.
n Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
* gdk/gdkpixbuf-drawable.c (G8fromRGB565): swap G and B.
Fixes (#114669)

17
ChangeLog.pre-2-8
View File

@@ -1,4 +1,19 @@
Sun Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
Fri Jun 6 11:07:33 2003 Owen Taylor <otaylor@redhat.com>
* gtk/gtkwidget.c (event_window_still_viewable):
Before delivering an event to a widget, check that
(if relevant), the event's window is still viewable.
(#105642, Dennis Björklund)
* gdk/gdkwindow.c (_gdk_window_destroy_hierarchy):
NULL out private->parent, since after destruction
it might not be valid any more.
* gdk/gdkwindow.c (gdk_window_is_viewable): Fix some
accesses before g_return_val_if_fail(). Treat
DESTROYED windows as unmapped.
n Jun 8 18:59:01 2003 Soeren Sandmann <sandmann@daimi.au.dk>
* gdk/gdkpixbuf-drawable.c (G8fromRGB565): swap G and B.
Fixes (#114669)

48
gtk/gtkwidget.c
View File

@@ -3038,12 +3038,60 @@ gtk_widget_send_expose (GtkWidget *widget,
return gtk_widget_event_internal (widget, event);
}
static gboolean
event_window_is_still_viewable (GdkEvent *event)
{
/* Check that we think the event's window is viewable before
* delivering the event, to prevent suprises. We do this here
* at the last moment, since the event may have been queued
* up behind other events, held over a recursive main loop, etc.
*/
switch (event->type)
{
case GDK_EXPOSE:
case GDK_MOTION_NOTIFY:
case GDK_BUTTON_PRESS:
case GDK_2BUTTON_PRESS:
case GDK_3BUTTON_PRESS:
case GDK_KEY_PRESS:
case GDK_ENTER_NOTIFY:
case GDK_PROXIMITY_IN:
case GDK_SCROLL:
return event->any.window && gdk_window_is_viewable (event->any.window);
#if 0
/* The following events are the second half of paired events;
* we always deliver them to deal with widgets that clean up
* on the second half.
*/
case GDK_BUTTON_RELEASE:
case GDK_KEY_RELEASE:
case GDK_LEAVE_NOTIFY:
case GDK_PROXIMITY_OUT:
#endif
default:
/* Remaining events would make sense on an not-viewable window,
* or don't have an associated window.
*/
return TRUE;
}
}
static gint
gtk_widget_event_internal (GtkWidget *widget,
GdkEvent *event)
{
gboolean return_val = FALSE;
/* We check only once for is-still-visible; if someone
* hides the window in on of the signals on the widget,
* they are responsible for returning TRUE to terminate
* handling.
*/
if (!event_window_is_still_viewable (event))
return TRUE;
g_object_ref (widget);
g_signal_emit (widget, widget_signals[EVENT], 0, event, &return_val);