[gnome-panel] Fix Force-Quit panel applet
- From: Jeremy Bicha <jbicha src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-panel] Fix Force-Quit panel applet
- Date: Mon, 20 May 2013 14:12:31 +0000 (UTC)
commit 5a8583a77aab91fe1a369becc9453b27bf30ee70
Author: Jeremy Bicha <jbicha ubuntu com>
Date: Mon May 20 10:10:34 2013 -0400
Fix Force-Quit panel applet
This patch implements handling of XInput2 extension events in the applet.
Without this patch, the applet fails to recognize such events (either mouse
click for killing application, or escape key for exiting), hence freezing the
desktop.
Also fix the event mask in call of gdk_device_grab() for keyboard.
Author: Sébastien Villemot <sebastien debian org>
http://bugs.debian.org/698740
gnome-panel/panel-force-quit.c | 34 +++++++++++++++++++++++++++-------
1 files changed, 27 insertions(+), 7 deletions(-)
---
diff --git a/gnome-panel/panel-force-quit.c b/gnome-panel/panel-force-quit.c
index fcf63a7..84594c6 100644
--- a/gnome-panel/panel-force-quit.c
+++ b/gnome-panel/panel-force-quit.c
@@ -32,6 +32,8 @@
#include <X11/Xlib.h>
#include <X11/keysym.h>
+#include <X11/extensions/XInput2.h>
+
#include "panel-icon-names.h"
#include "panel-stock-icons.h"
@@ -246,22 +248,23 @@ kill_window_question (gpointer window)
static void
handle_button_press_event (GtkWidget *popup,
- XKeyEvent *event)
+ Display *display,
+ Window subwindow)
{
Window window;
remove_popup (popup);
- if (event->subwindow == None)
+ if (subwindow == None)
return;
if (wm_state_atom == None)
- wm_state_atom = XInternAtom (event->display, "WM_STATE", FALSE);
+ wm_state_atom = XInternAtom (display, "WM_STATE", FALSE);
- window = find_managed_window (event->display, event->subwindow);
+ window = find_managed_window (display, subwindow);
if (window != None) {
- if (!gdk_x11_window_lookup_for_display (gdk_x11_lookup_xdisplay (event->display), window))
+ if (!gdk_x11_window_lookup_for_display (gdk_x11_lookup_xdisplay (display), window))
kill_window_question ((gpointer) window);
}
}
@@ -272,10 +275,12 @@ popup_filter (GdkXEvent *gdk_xevent,
GtkWidget *popup)
{
XEvent *xevent = (XEvent *) gdk_xevent;
+ XIEvent *xiev;
+ XIDeviceEvent *xidev;
switch (xevent->type) {
case ButtonPress:
- handle_button_press_event (popup, &xevent->xkey);
+ handle_button_press_event (popup, xevent->xbutton.display, xevent->xbutton.subwindow);
return GDK_FILTER_REMOVE;
case KeyPress:
if (xevent->xkey.keycode == XKeysymToKeycode (xevent->xany.display, XK_Escape)) {
@@ -283,6 +288,21 @@ popup_filter (GdkXEvent *gdk_xevent,
return GDK_FILTER_REMOVE;
}
break;
+ case GenericEvent:
+ xiev = (XIEvent *) xevent->xcookie.data;
+ xidev = (XIDeviceEvent *) xiev;
+ switch (xiev->evtype) {
+ case XI_KeyPress:
+ if (xidev->detail == XKeysymToKeycode (xevent->xany.display, XK_Escape)) {
+ remove_popup (popup);
+ return GDK_FILTER_REMOVE;
+ }
+ break;
+ case XI_ButtonPress:
+ handle_button_press_event (popup, xidev->display, xidev->child);
+ return GDK_FILTER_REMOVE;
+ }
+ break;
default:
break;
}
@@ -331,7 +351,7 @@ panel_force_quit (GdkScreen *screen,
status = gdk_device_grab (keyboard, root,
GDK_OWNERSHIP_NONE, FALSE,
- GDK_KEY_PRESS | GDK_KEY_RELEASE,
+ GDK_KEY_PRESS_MASK | GDK_KEY_RELEASE_MASK,
NULL, time);
if (status != GDK_GRAB_SUCCESS) {
g_warning ("Keyboard grab failed\n");
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]