Get window position by reading _NET_FRAME_EXTENTS (Bug#5721).

* src/xfns.c (x_real_positions): Try to get _NET_FRAME_EXTENTS first
before traversing window tree (Bug#5721).

* src/xterm.c (x_term_init): Initialize Xatom_net_frame_extents.

* src/xterm.h (struct x_display_info): Xatom_net_frame_extents is new.

diff --git a/src/ChangeLog b/src/ChangeLog
index a44fdbb7e4295887f51be0448e35a688bfcf7faf..a1a049c66c1485effb4c99097eff66d535cb40a4 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,12 @@
+2010-11-04  Jan Djärv  <jan.h.d@swipnet.se>
+
+       * xterm.c (x_term_init): Initialize Xatom_net_frame_extents.
+
+       * xterm.h (struct x_display_info): Xatom_net_frame_extents is new.
+
+       * xfns.c (x_real_positions): Try to get _NET_FRAME_EXTENTS first
+       before traversing window tree (Bug#5721).
+
 2010-11-03  Glenn Morris  <rgm@gnu.org>
 
        * Makefile.in (CRT_DIR): New variable, set by configure.

diff --git a/src/xfns.c b/src/xfns.c
index 1d7d3d0358029cafae00f79259c3be1198a6c5b3..0b0f1cd0d96f364da3875a1cb39de82f3bca8a81 100644 (file)
--- a/src/xfns.c
+++ b/src/xfns.c
@@ -532,12 +532,60 @@ x_real_positions (f, xptr, yptr)
   int win_x, win_y, outer_x, outer_y;
   int real_x = 0, real_y = 0;
   int had_errors = 0;
-  Window win = f->output_data.x->parent_desc;
+  Window win;
+  Atom actual_type;
+  unsigned long actual_size, bytes_remaining;
+  int i, rc, actual_format;
+  struct x_display_info *dpyinfo = FRAME_X_DISPLAY_INFO (f);
+  long max_len = 400;
+  Display *dpy = FRAME_X_DISPLAY (f);
+  unsigned char *tmp_data = NULL;
+  Atom target_type = XA_CARDINAL;
 
   BLOCK_INPUT;
 
-  x_catch_errors (FRAME_X_DISPLAY (f));
+  x_catch_errors (dpy);
+
+  win = FRAME_OUTER_WINDOW (f);
+  /* Try _NET_FRAME_EXTENTS first.  */
+  rc = XGetWindowProperty (dpy, win, dpyinfo->Xatom_net_frame_extents,
+                           0, max_len, False, target_type,
+                           &actual_type, &actual_format, &actual_size,
+                           &bytes_remaining, &tmp_data);
+
+  if (0 && rc == Success && actual_type == target_type && !x_had_errors_p (dpy)
+      && actual_size == 4 && actual_format == 32)
+    {
+      int ign;
+      Window rootw;
+
+      XGetGeometry (FRAME_X_DISPLAY (f), win,
+                    &rootw, &real_x, &real_y, &ign, &ign, &ign, &ign);
+      long *fe = (long *)tmp_data;
+      
+      FRAME_X_OUTPUT (f)->x_pixels_outer_diff = fe[0];
+      FRAME_X_OUTPUT (f)->y_pixels_outer_diff = fe[2];
+      *xptr = real_x - fe[0];
+      *yptr = real_y - fe[2];
+
+      if (FRAME_X_WINDOW (f) != win) 
+        {
+          XGetGeometry (FRAME_X_DISPLAY (f), FRAME_X_WINDOW (f),
+                        &rootw, &real_x, &real_y, &ign, &ign, &ign, &ign);
+
+          f->x_pixels_diff = real_x;
+          f->y_pixels_diff = real_y;
+        }
+
+      if (tmp_data) XFree (tmp_data);
+      x_uncatch_errors ();
+      UNBLOCK_INPUT;
+      return;
+    }
+
+  if (tmp_data) XFree (tmp_data);
 
+  win = f->output_data.x->parent_desc;
   if (win == FRAME_X_DISPLAY_INFO (f)->root_window)
     win = FRAME_OUTER_WINDOW (f);
 

diff --git a/src/xterm.c b/src/xterm.c
index 808eaad3f5f7d9008f7d5dfc791de176244f3263..22019e2279b9ced2d9c5f51e83acac6c71116b7d 100644 (file)
--- a/src/xterm.c
+++ b/src/xterm.c
@@ -10522,7 +10522,8 @@ x_term_init (display_name, xrm_option, resource_name)
     = XInternAtom (dpyinfo->display, "_NET_WM_WINDOW_TYPE", False);
   dpyinfo->Xatom_net_window_type_tooltip
     = XInternAtom (dpyinfo->display, "_NET_WM_WINDOW_TYPE_TOOLTIP", False);
-  
+  dpyinfo->Xatom_net_frame_extents  
+    = XInternAtom (dpyinfo->display, "_NET_FRAME_EXTENTS", False);
   dpyinfo->cut_buffers_initialized = 0;
 
   dpyinfo->x_dnd_atoms_size = 8;

diff --git a/src/xterm.h b/src/xterm.h
index c8601b8c43dc4f6dc53ced82d4c84e5f6732f3e4..07eaec3060cd4cadee56a5241f2b74bba43dfc9b 100644 (file)
--- a/src/xterm.h
+++ b/src/xterm.h
@@ -362,10 +362,10 @@ struct x_display_info
   Window net_supported_window;
   Atom Xatom_net_window_type, Xatom_net_window_type_tooltip;
 
-  /* Atoms dealing with maximization and fullscreen */
+  /* Atoms dealing with EWMH (i.e. _NET_...) */
   Atom Xatom_net_wm_state, Xatom_net_wm_state_fullscreen_atom,
     Xatom_net_wm_state_maximized_horz, Xatom_net_wm_state_maximized_vert,
-    Xatom_net_wm_state_sticky;
+    Xatom_net_wm_state_sticky, Xatom_net_frame_extents;
 
   /* XSettings atoms and windows.  */
   Atom Xatom_xsettings_sel, Xatom_xsettings_prop, Xatom_xsettings_mgr;