ndk/native_window: Implement HasWindowHandle for 0.6 (and maintain …

…0.5)

Following [the same strategy in winit], move `raw-window-handle 0.5`
behind a `rwh_05` feature and add `raw-window-handle 0.6` behind a
corresponding `rwh_06` feature.  Implement the new non-raw trait and
**lifetimed** type for `0.6` (where `raw-window-handle` provides blanket
implementations to coerce/convert back to `Raw*` types).

[the same strategy in winit]: rust-windowing/winit#3126

ndk/Cargo.toml

@@ -13,7 +13,7 @@ repository = "https://github.com/rust-mobile/ndk"
 rust-version = "1.66"
 
 [features]
-all = ["audio", "bitmap","media", "api-level-31"]
+all = ["audio", "bitmap","media", "api-level-31", "rwh_05", "rwh_06"]
 
 audio = ["ffi/audio", "api-level-26"]
 bitmap = ["ffi/bitmap"]
@@ -36,7 +36,8 @@ bitflags = "2.0.0"
 jni-sys = "0.3.0"
 log = "0.4"
 num_enum = "0.7"
-raw-window-handle = "0.5"
+rwh_05 = { package = "raw-window-handle", version = "0.5", optional = true }
+rwh_06 = { package = "raw-window-handle", version = "0.6", optional = true }
 thiserror = "1.0.23"
 
 [dependencies.jni]

ndk/src/native_window.rs

@@ -6,8 +6,7 @@ use crate::utils::status_to_io_result;
 
 pub use super::hardware_buffer_format::HardwareBufferFormat;
 use jni_sys::{jobject, JNIEnv};
-use raw_window_handle::{AndroidNdkWindowHandle, HasRawWindowHandle, RawWindowHandle};
-use std::{ffi::c_void, io::Result, mem::MaybeUninit, ptr::NonNull};
+use std::{ffi::c_void, io, mem::MaybeUninit, ptr::NonNull};
 
 pub type Rect = ffi::ARect;
 
@@ -39,11 +38,22 @@ impl Clone for NativeWindow {
     }
 }
 
-unsafe impl HasRawWindowHandle for NativeWindow {
-    fn raw_window_handle(&self) -> RawWindowHandle {
-        let mut handle = AndroidNdkWindowHandle::empty();
-        handle.a_native_window = self.ptr.as_ptr() as *mut c_void;
-        RawWindowHandle::AndroidNdk(handle)
+#[cfg(feature = "rwh_05")]
+unsafe impl rwh_05::HasRawWindowHandle for NativeWindow {
+    fn raw_window_handle(&self) -> rwh_05::RawWindowHandle {
+        let mut handle = rwh_05::AndroidNdkWindowHandle::empty();
+        handle.a_native_window = self.ptr.as_ptr().cast();
+        rwh_05::RawWindowHandle::AndroidNdk(handle)
+    }
+}
+
+#[cfg(feature = "rwh_06")]
+impl rwh_06::HasWindowHandle for NativeWindow {
+    fn window_handle(&self) -> Result<rwh_06::WindowHandle<'_>, rwh_06::HandleError> {
+        let handle = rwh_06::AndroidNdkWindowHandle::new(self.ptr.cast());
+        let handle = rwh_06::RawWindowHandle::AndroidNdk(handle);
+        // SAFETY: All fields of the "raw"  `AndroidNdkWindowHandle` struct are filled out
+        Ok(unsafe { rwh_06::WindowHandle::borrow_raw(handle) })
     }
 }
 
@@ -98,7 +108,7 @@ impl NativeWindow {
         width: i32,
         height: i32,
         format: Option<HardwareBufferFormat>,
-    ) -> Result<()> {
+    ) -> io::Result<()> {
         let format = format.map_or(0, |f| {
             u32::from(f)
                 .try_into()
@@ -112,7 +122,7 @@ impl NativeWindow {
 
     /// Set a transform that will be applied to future buffers posted to the window.
     #[cfg(feature = "api-level-26")]
-    pub fn set_buffers_transform(&self, transform: NativeWindowTransform) -> Result<()> {
+    pub fn set_buffers_transform(&self, transform: NativeWindowTransform) -> io::Result<()> {
         let status = unsafe {
             ffi::ANativeWindow_setBuffersTransform(self.ptr.as_ptr(), transform.bits() as i32)
         };
@@ -133,7 +143,7 @@ impl NativeWindow {
         &self,
         frame_rate: f32,
         compatibility: FrameRateCompatibility,
-    ) -> Result<()> {
+    ) -> io::Result<()> {
         let compatibility = (compatibility as u32)
             .try_into()
             .expect("i8 overflow in FrameRateCompatibility");
@@ -180,7 +190,7 @@ impl NativeWindow {
         frame_rate: f32,
         compatibility: FrameRateCompatibility,
         change_frame_rate_strategy: ChangeFrameRateStrategy,
-    ) -> Result<()> {
+    ) -> io::Result<()> {
         let compatibility = (compatibility as u32)
             .try_into()
             .expect("i8 overflow in FrameRateCompatibility");
@@ -235,7 +245,10 @@ impl NativeWindow {
     ///
     /// Optionally pass the region you intend to draw into `dirty_bounds`.  When this function
     /// returns it is updated (commonly enlarged) with the actual area the caller needs to redraw.
-    pub fn lock(&self, dirty_bounds: Option<&mut Rect>) -> Result<NativeWindowBufferLockGuard<'_>> {
+    pub fn lock(
+        &self,
+        dirty_bounds: Option<&mut Rect>,
+    ) -> io::Result<NativeWindowBufferLockGuard<'_>> {
         let dirty_bounds = match dirty_bounds {
             Some(dirty_bounds) => dirty_bounds,
             None => std::ptr::null_mut(),