From 6ab514c4280f5244829705e23c45f75a724d22d9 Mon Sep 17 00:00:00 2001
From: Lucas Schwiderski <lucas@lschwiderski.de>
Date: Fri, 24 Nov 2023 11:07:40 +0100
Subject: [PATCH] dtmm: Filter file dialog error log

Druid's implementation makes it so that cancelling the file dialog
logs an error tracing event.

Closes #133.
---
 crates/dtmm/src/util/log.rs | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/crates/dtmm/src/util/log.rs b/crates/dtmm/src/util/log.rs
index 09fc9a6..fa4a643 100644
--- a/crates/dtmm/src/util/log.rs
+++ b/crates/dtmm/src/util/log.rs
@@ -56,7 +56,7 @@ impl std::io::Write for ChannelWriter {
 }
 
 pub fn create_tracing_subscriber(tx: UnboundedSender<Vec<u8>>, level: Option<LogLevel>) {
-    let env_layer = if let Some(level) = level {
+    let mut env_layer = if let Some(level) = level {
         EnvFilter::from(level)
     } else if cfg!(debug_assertions) {
         EnvFilter::try_from_default_env().unwrap_or_else(|_| EnvFilter::new("info"))
@@ -64,6 +64,18 @@ pub fn create_tracing_subscriber(tx: UnboundedSender<Vec<u8>>, level: Option<Log
         EnvFilter::new("error,dtmm=info")
     };
 
+    // The internal implementation of Druid's GTK file dialog turns
+    // cancelling the dialog into an error. The, also internal, wrapper
+    // then logs and swallows the error.
+    // Therefore, as a consumer of the library, we don't have any way
+    // to customize this behavior, and instead have to filter out the
+    // tracing event.
+    env_layer = env_layer.add_directive(
+        "druid_shell::backend::gtk::window=off"
+            .parse()
+            .expect("Invalid env filter directive"),
+    );
+
     let stdout_layer = fmt::layer().pretty();
 
     let channel_layer = fmt::layer()