From 91b47a3188f92fa36cb838d41310067d5e000174 Mon Sep 17 00:00:00 2001
From: Aaron Muir Hamilton <aaron@correspondwith.me>
Date: Wed, 23 Oct 2024 17:10:54 -0400
Subject: [PATCH] vello_editor: Use PresentMode::Mailbox when available. (#144)

AutoVsync/FIFO introduces unnecessary latency, especially on X11. There
aren't animations in vello_editor so the downside (extra work) is not
significant.
---
 examples/vello_editor/src/main.rs | 34 +++++++++++++++++++++++--------
 1 file changed, 26 insertions(+), 8 deletions(-)

diff --git a/examples/vello_editor/src/main.rs b/examples/vello_editor/src/main.rs
index f59c118c..339580b4 100644
--- a/examples/vello_editor/src/main.rs
+++ b/examples/vello_editor/src/main.rs
@@ -63,15 +63,7 @@ impl ApplicationHandler for SimpleVelloApp<'_> {
             .take()
             .unwrap_or_else(|| create_winit_window(event_loop));
 
-        // Create a vello Surface
         let size = window.inner_size();
-        let surface_future = self.context.create_surface(
-            window.clone(),
-            size.width,
-            size.height,
-            wgpu::PresentMode::AutoVsync,
-        );
-        let surface = pollster::block_on(surface_future).expect("Error creating surface");
 
         self.editor.transact([
             PlainEditorOp::SetScale(1.0),
@@ -79,9 +71,35 @@ impl ApplicationHandler for SimpleVelloApp<'_> {
             PlainEditorOp::SetText(text::LOREM.into()),
         ]);
 
+        // Create a vello Surface
+        let surface_future = {
+            let surface = self
+                .context
+                .instance
+                .create_surface(wgpu::SurfaceTarget::from(window.clone()))
+                .expect("Error creating surface");
+            let dev_id = pollster::block_on(self.context.device(Some(&surface)))
+                .expect("No compatible device");
+            let device_handle = &self.context.devices[dev_id];
+            let capabilities = surface.get_capabilities(device_handle.adapter());
+            let present_mode = if capabilities
+                .present_modes
+                .contains(&wgpu::PresentMode::Mailbox)
+            {
+                wgpu::PresentMode::Mailbox
+            } else {
+                wgpu::PresentMode::AutoVsync
+            };
+
+            self.context
+                .create_render_surface(surface, size.width, size.height, present_mode)
+        };
+        let surface = pollster::block_on(surface_future).expect("Error creating surface");
+
         // Create a vello Renderer for the surface (using its device id)
         self.renderers
             .resize_with(self.context.devices.len(), || None);
+
         self.renderers[surface.dev_id]
             .get_or_insert_with(|| create_vello_renderer(&self.context, &surface));