commit 71c3877a401bab2e1c19c5d84cb9a8e28ff0895d
author Daniel Stone <daniel@fooishbar.org> Fri Jun 22 13:21:30 2012 +0100
committer Kristian Høgsberg <krh@bitplanet.net> Fri Jun 22 11:52:05 2012 -0400
tree 0460b6b502d1928cf1915446e558cbe4e7c2761a
parent d6da09e9353ca64990daaa70b5226dba730977a0

Let wayland-server send modifier events for focus

When the focus changes, wl_keyboard_set_focus (and, as an added bonus,
wl_pointer_set_focus) will now send wl_keyboard::modifier events for us
if we store the modifier state in the right place, so we don't have to
worry about that anymore.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

src/compositor.c

diff --git a/src/compositor.c b/src/compositor.c
index 9d4d467..d914b67 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -1688,16 +1688,6 @@
 	if (seat->seat.keyboard) {
 		wl_keyboard_set_focus(seat->seat.keyboard, &surface->surface);
 		wl_data_device_set_keyboard_focus(&seat->seat);
-
-		if (seat->seat.keyboard->focus_resource) {
-			wl_keyboard_send_modifiers(
-				seat->seat.keyboard->focus_resource,
-				wl_display_next_serial(compositor->wl_display),
-				seat->xkb_state.mods_depressed,
-				seat->xkb_state.mods_latched,
-				seat->xkb_state.mods_locked,
-				seat->xkb_state.group);
-		}
 	}
 
 	wl_signal_emit(&compositor->activate_signal, surface);
@@ -1785,16 +1775,16 @@
 	group = xkb_state_serialize_group(seat->xkb_state.state,
 					  XKB_STATE_EFFECTIVE);
 
-	if (mods_depressed != seat->xkb_state.mods_depressed ||
-	    mods_latched != seat->xkb_state.mods_latched ||
-	    mods_locked != seat->xkb_state.mods_locked ||
-	    group != seat->xkb_state.group)
+	if (mods_depressed != seat->seat.keyboard->modifiers.mods_depressed ||
+	    mods_latched != seat->seat.keyboard->modifiers.mods_latched ||
+	    mods_locked != seat->seat.keyboard->modifiers.mods_locked ||
+	    group != seat->seat.keyboard->modifiers.group)
 		ret = 1;
 
-	seat->xkb_state.mods_depressed = mods_depressed;
-	seat->xkb_state.mods_latched = mods_latched;
-	seat->xkb_state.mods_locked = mods_locked;
-	seat->xkb_state.group = group;
+	seat->seat.keyboard->modifiers.mods_depressed = mods_depressed;
+	seat->seat.keyboard->modifiers.mods_latched = mods_latched;
+	seat->seat.keyboard->modifiers.mods_locked = mods_locked;
+	seat->seat.keyboard->modifiers.group = group;
 
 	/* And update the modifier_state for bindings. */
 	mods_lookup = mods_depressed | mods_latched;
@@ -1887,10 +1877,10 @@
 	if (mods)
 		grab->interface->modifiers(grab,
 					   wl_display_get_serial(compositor->wl_display),
-					   ws->xkb_state.mods_depressed,
-					   ws->xkb_state.mods_latched,
-					   ws->xkb_state.mods_locked,
-					   ws->xkb_state.group);
+					   seat->keyboard->modifiers.mods_depressed,
+					   seat->keyboard->modifiers.mods_latched,
+					   seat->keyboard->modifiers.mods_locked,
+					   seat->keyboard->modifiers.group);
 }
 
 WL_EXPORT void
@@ -1948,15 +1938,6 @@
 	if (surface) {
 		wl_list_remove(&ws->saved_kbd_focus_listener.link);
 		wl_keyboard_set_focus(ws->seat.keyboard, surface);
-
-		if (seat->keyboard->focus_resource) {
-			wl_keyboard_send_modifiers(seat->keyboard->focus_resource,
-						   wl_display_next_serial(compositor->wl_display),
-						   ws->xkb_state.mods_depressed,
-						   ws->xkb_state.mods_latched,
-						   ws->xkb_state.mods_locked,
-						   ws->xkb_state.group);
-		}
 		ws->saved_kbd_focus = NULL;
 	}
 }
@@ -2482,10 +2463,7 @@
 		exit(1);
 	}
 
-	seat->xkb_state.mods_depressed = 0;
-	seat->xkb_state.mods_latched = 0;
-	seat->xkb_state.mods_locked = 0;
-	seat->xkb_state.group = 0;
+	seat->xkb_state.leds = 0;
 
 	wl_keyboard_init(&seat->keyboard);
 	wl_seat_set_keyboard(&seat->seat, &seat->keyboard);