feat: support create region from winit

Author: SergioRibera

src/platform/wayland.rs

@@ -14,9 +14,13 @@
 //! * `wayland-csd-adwaita-crossfont`.
 //! * `wayland-csd-adwaita-notitle`.
 #[cfg(wayland_platform)]
-use sctk::shell::wlr_layer::{Anchor, KeyboardInteractivity, Layer};
+pub use sctk::{
+    compositor::Region,
+    shell::wlr_layer::{Anchor, KeyboardInteractivity, Layer},
+};
 
 use crate::dpi::{LogicalPosition, LogicalSize};
+use crate::error::RequestError;
 use crate::event_loop::{ActiveEventLoop, EventLoop, EventLoopBuilder};
 use crate::monitor::MonitorHandle;
 pub use crate::window::Theme;
@@ -78,11 +82,27 @@ impl EventLoopBuilderExtWayland for EventLoopBuilder {
 ///
 /// [`Window`]: crate::window::Window
 pub trait WindowExtWayland {
-    fn set_region(&self, region: Option<(LogicalPosition<i32>, LogicalSize<i32>)>);
+    fn create_region(
+        &self,
+        position: LogicalPosition<i32>,
+        size: LogicalSize<i32>,
+    ) -> Result<Region, RequestError>;
+    fn set_region(&self, region: Option<&Region>);
 }
 
 impl WindowExtWayland for dyn CoreWindow + '_ {
-    fn set_region(&self, region: Option<(LogicalPosition<i32>, LogicalSize<i32>)>) {
+    fn create_region(
+        &self,
+        pos: LogicalPosition<i32>,
+        size: LogicalSize<i32>,
+    ) -> Result<Region, RequestError> {
+        let window = self.as_any().downcast_ref::<crate::platform_impl::wayland::Window>().unwrap();
+        let region = window.create_region()?;
+        region.add(pos.x, pos.y, size.width, size.height);
+        Ok(region)
+    }
+
+    fn set_region(&self, region: Option<&Region>) {
         let window = self.as_any().downcast_ref::<crate::platform_impl::wayland::Window>().unwrap();
         window.set_region(region);
     }

src/platform_impl/linux/wayland/window/mod.rs

@@ -335,16 +335,15 @@ impl Window {
     }
 
     #[inline]
-    pub fn set_region(&self, region: Option<(LogicalPosition<i32>, LogicalSize<i32>)>) {
+    pub fn create_region(&self) -> Result<Region, RequestError> {
+        Ok(Region::new(&*self.compositor).map_err(|err| os_error!(err))?)
+    }
+
+    #[inline]
+    pub fn set_region(&self, region: Option<&Region>) {
         match &self.window {
             WindowShell::WlrLayer { surface } => {
-                if let Some((pos, size)) = region {
-                    let region = Region::new(self.compositor.as_ref()).unwrap();
-                    region.add(pos.x, pos.y, size.width, size.height);
-                    surface.set_input_region(Some(region.wl_region()));
-                } else {
-                    surface.set_input_region(None);
-                }
+                surface.set_input_region(region.map(|r| r.wl_region()));
             },
             WindowShell::Xdg { .. } => warn!("Region is ignored for XDG windows"),
         }