From: Tony Crisci <tony@dubstepdish.com>
Date: Sat, 29 Mar 2014 04:25:52 +0000 (-0400)
Subject: Send last event timestamp with WM_TAKE_FOCUS msg
X-Git-Tag: 4.8~88
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=4ded44d18a4dfe1f2bcad6ee0da121ab570d5152;p=i3%2Fi3

Send last event timestamp with WM_TAKE_FOCUS msg

According to 4.1.7 of the iccm spec

http://tronche.com/gui/x/icccm/sec-4.html#s-4.1.7

> Windows with the atom WM_TAKE_FOCUS in their WM_PROTOCOLS property may
> receive a ClientMessage event from the window manager (as described in
> section 4.2.8) with WM_TAKE_FOCUS in its data[0] field and a valid
> timestamp (i.e. not CurrentTime ) in its data[1] field.

Adds the timestamp parameter to send_take_focus to avoid the dangerous
use of a global variable.
---

diff --git a/include/xcb.h b/include/xcb.h
index 848fe295..4df7f639 100644
--- a/include/xcb.h
+++ b/include/xcb.h
@@ -94,7 +94,7 @@ void fake_absolute_configure_notify(Con *con);
  * Sends the WM_TAKE_FOCUS ClientMessage to the given window
  *
  */
-void send_take_focus(xcb_window_t window);
+void send_take_focus(xcb_window_t window, xcb_timestamp_t timestamp);
 
 /**
  * Raises the given window (typically client->frame) above all other windows
diff --git a/src/x.c b/src/x.c
index cd36a283..eca95da0 100644
--- a/src/x.c
+++ b/src/x.c
@@ -992,7 +992,7 @@ void x_push_changes(Con *con) {
                 focused->window->needs_take_focus) {
                 DLOG("Updating focus by sending WM_TAKE_FOCUS to window 0x%08x (focused: %p / %s)\n",
                      to_focus, focused, focused->name);
-                send_take_focus(to_focus);
+                send_take_focus(to_focus, last_timestamp);
                 set_focus = !focused->window->doesnt_accept_focus;
                 DLOG("set_focus = %d\n", set_focus);
 
diff --git a/src/xcb.c b/src/xcb.c
index caa203f7..3039b030 100644
--- a/src/xcb.c
+++ b/src/xcb.c
@@ -110,7 +110,7 @@ void fake_absolute_configure_notify(Con *con) {
  * Sends the WM_TAKE_FOCUS ClientMessage to the given window
  *
  */
-void send_take_focus(xcb_window_t window) {
+void send_take_focus(xcb_window_t window, xcb_timestamp_t timestamp) {
     /* Every X11 event is 32 bytes long. Therefore, XCB will copy 32 bytes.
      * In order to properly initialize these bytes, we allocate 32 bytes even
      * though we only need less for an xcb_configure_notify_event_t */
@@ -122,7 +122,7 @@ void send_take_focus(xcb_window_t window) {
     ev->type = A_WM_PROTOCOLS;
     ev->format = 32;
     ev->data.data32[0] = A_WM_TAKE_FOCUS;
-    ev->data.data32[1] = XCB_CURRENT_TIME;
+    ev->data.data32[1] = timestamp;
 
     DLOG("Sending WM_TAKE_FOCUS to the client\n");
     xcb_send_event(conn, false, window, XCB_EVENT_MASK_NO_EVENT, (char*)ev);