implement get_position for x11 and wayland

window/src/os/wayland/window.rs

@@ -1017,6 +1017,15 @@ impl WindowOps for WaylandWindow {
             Ok(())
         });
     }
+
+    fn get_window_position(&self) -> Future<ScreenPoint> {
+        WaylandConnection::with_window_inner(self.0, move |_inner| {
+            // Wayland does not support getting window position. Potential workarounds:
+            //   1. Use window manager like GTK
+            //   2. Create window covering the all screens and use subsurfaces
+            Ok(ScreenPoint::new(0, 0))
+        })
+    }
 }
 
 pub(crate) fn read_pipe_with_timeout(mut file: FileDescriptor) -> anyhow::Result<String> {

window/src/os/x11/window.rs

@@ -1397,16 +1397,15 @@ impl XWindowInner {
         });
     }
 
-    fn get_window_position(&mut self) -> ScreenPoint {
-        let conn = self.conn();
+    /// Get position of the top left corner of client region of the window.
+    fn get_window_position(&mut self) -> anyhow::Result<ScreenPoint> {
         let geom = self
             .conn()
             .send_and_wait_request(&xcb::x::GetGeometry {
                 drawable: xcb::x::Drawable::Window(self.window_id),
             })
             .context("querying geometry")?;
-        log::trace!("window position is x={} y={}", geom.x(), geom.y(),);
-        ScreenPoint::new(geom.x(), geom.y())
+        Ok(ScreenPoint::new(geom.x().into(), geom.y().into()))
     }
 
     /// Change the title for the window manager
@@ -1686,6 +1685,10 @@ impl WindowOps for XWindow {
         });
     }
 
+    fn get_window_position(&self) -> Future<ScreenPoint> {
+        XConnection::with_window_inner(self.0, move |inner| inner.get_window_position())
+    }
+
     fn set_window_position(&self, coords: ScreenPoint) {
         XConnection::with_window_inner(self.0, move |inner| {
             inner.set_window_position(coords);

window/src/os/x_and_wayland.rs

@@ -386,11 +386,11 @@ impl WindowOps for Window {
         }
     }
 
-    fn get_window_position(&self) -> ScreenPoint {
+    fn get_window_position(&self) -> Future<ScreenPoint> {
         match self {
             Self::X11(x) => x.get_window_position(),
             #[cfg(feature = "wayland")]
-            Self::Wayland(w) => ScreenPoint(0, 0),
+            Self::Wayland(w) => w.get_window_position(),
         }
     }