mirror of
https://gitlab.uni-freiburg.de/opensourcevdi/spice
synced 2025-12-27 07:29:32 +00:00
86e62813b3
Currently modifier keys (ctrl, alt) can get stuck when using the x11 client. To reproduce under gnome: -focus the client window without causing it to grab the keyborad (click on the title bar not the window) -press crlt + alt + right arrow to switch virtual desktop -press crlt + alt + left arrow to switch back -notice ctrl + alt are stuck pressed What is happening here is: -We get a focus out event, caused by the hotkey combi key grab, focus event notify mode == NotifyGrab, and release all keys -> good -We get another focus out event, as we really loose the focus. notify mode == NotifyWhileGrabbed, which we ignore as we already lost focus before -We get a focus in event, as the focus is returning to us, but we don't really have the focus yet, as the hotkey combi key grab is still active (ie ctrl + alt are still pressed). We now sync the vm's modifier key state with the current X-server state, telling the vm ctrl + alt are pressed. Note we do this by directly reading the X-server keyboard status, we are not getting any key press events from the X-server -> bad -We get another focus in event, as we really get the focus back, notify mode == NotifyUngrab. We ignore this one as already have gained the focus before. If we were to sync the vm modifier state here, all would be well we would no longer see the modifier keys pressed, or if we would we would get a release event when they get released (testing has shown both). The solution here is to ignore the first focus in event, and do the modifier sync on the second focus in event, or more in general to ignore focus events where notify mode == NotifyWhileGrabbed. |
||
|---|---|---|
| .. | ||
| images | ||
| .gitignore | ||
| atomic_count.h | ||
| event_sources_p.cpp | ||
| event_sources_p.h | ||
| main.cpp | ||
| Makefile.am | ||
| named_pipe.cpp | ||
| named_pipe.h | ||
| pixels_source_p.h | ||
| pixels_source.cpp | ||
| platform_utils.cpp | ||
| platform_utils.h | ||
| platform.cpp | ||
| playback.cpp | ||
| playback.h | ||
| record.cpp | ||
| record.h | ||
| red_drawable.cpp | ||
| red_pixmap_gl.cpp | ||
| red_pixmap_sw.cpp | ||
| red_pixmap.cpp | ||
| red_window_p.h | ||
| red_window.cpp | ||
| res.cpp | ||
| res.h | ||
| resource.h | ||
| x_icon.cpp | ||
| x_icon.h | ||
| x_platform.h | ||