gdk: do not deactivate surface on keyboard grabs
authorSamuel Thibault <samuel.thibault@ens-lyon.org>
Thu, 19 Apr 2018 12:10:23 +0000 (14:10 +0200)
committerSamuel Thibault <samuel.thibault@ens-lyon.org>
Mon, 30 Apr 2018 08:57:53 +0000 (10:57 +0200)
commit01455399e83a3dbafb0cdc6e12c7003b2f472a40
tree8eadc5657b4ed8fe2cb448c51e51a23df07c06de
parent3684b7212150d277fd592a38f6f742019456036f
gdk: do not deactivate surface on keyboard grabs

When pressing e.g. a window manager shortcut, which acquires keyboard grab,
Xorg would send FocusOut NotifyGrab then FocusIn NotifyUngrab.  Currently
gdk would then deactivate the current surface, which makes accessibility
screen readers think that we have switched to a non-accessible application
and came back again, and thus reannounce the application frame etc. which we
don't want when e.g. just raising volume.

And actually, receiving FocusOut NotifyGrab does not mean losing the
X focus, it only means an application aqcuired a grab, i.e. it is
temporarily stealing keyboard events. On Wayland, this isn't even
notified actually.

This commit makes gdk only deactivate surfaces when there was an actual
focus switch to another window, as determined by has_focus_window (instead
of just has_focus), which happens either normally through FocusOut with
NotifyNormal, or during grabs through FocusOut with NotifyWhileGrabbed.

Fixes #85
gdk/x11/gdkdevicemanager-core-x11.c
gdk/x11/gdkeventsource.c