Kaynağa Gözat

fix(core): clear resource table in `cleanup_before_exit` (#9505)

* Clear resource table in `cleanup_before_exit`

* Add change file

* Update .changes/cleanup-resource-table.md
Tony 1 yıl önce
ebeveyn
işleme
daf018e4f5

+ 5 - 0
.changes/cleanup-resource-table.md

@@ -0,0 +1,5 @@
+---
+tauri: patch:bug
+---
+
+Fix resource tables not cleaned up on exit which causes tray icon inside resource tables not cleaned up on exit

+ 8 - 2
core/tauri/src/app.rs

@@ -754,6 +754,13 @@ macro_rules! shared_app_impl {
       pub fn cleanup_before_exit(&self) {
         #[cfg(all(desktop, feature = "tray-icon"))]
         self.manager.tray.icons.lock().unwrap().clear();
+        self.manager.resources_table().clear();
+        for (_, window) in self.manager.windows().iter() {
+          window.resources_table().clear();
+        }
+        for (_, webview) in self.manager.webviews().iter() {
+          webview.resources_table().clear();
+        }
       }
     }
 
@@ -1700,8 +1707,7 @@ tauri::Builder::default()
         if let Some(tooltip) = &tray_config.tooltip {
           tray = tray.tooltip(tooltip);
         }
-        let tray = tray.build(handle)?;
-        app.manager.tray.icons.lock().unwrap().push(tray);
+        tray.build(handle)?;
       }
     }
 

+ 6 - 0
core/tauri/src/resources/mod.rs

@@ -209,4 +209,10 @@ impl ResourceTable {
       .ok_or_else(|| crate::Error::BadResourceId(rid))
       .map(|resource| resource.close())
   }
+
+  /// Removes and frees all resources stored. Note that the
+  /// resource's `close()` method is *not* called.
+  pub(crate) fn clear(&mut self) {
+    self.index.clear()
+  }
 }