fix: remove Rust warnings in browser commands and record browser visits

src-tauri/src/browser/commands.rs

@@ -2,7 +2,7 @@ use std::collections::HashMap;
 use tauri::webview::{PageLoadEvent, WebviewBuilder};
 use tauri::{AppHandle, LogicalPosition, LogicalSize, Manager, Url, WebviewUrl};
 
-use super::layout::{bounds, hide_tab, resize_active, show_tab};
+use super::layout::{hide_tab, resize_active, show_tab};
 use super::navigation::{navigation_blocked, resolve_url};
 use super::tab_manager::*;
 use crate::adblock::engine::AdBlockState;
@@ -51,18 +51,10 @@ pub async fn browser_set_visible(app: AppHandle, visible: bool, layout: Viewport
     Ok(())
 }
 
-/// Hard hide every child browser webview. Use this when the browser drawer closes.
-/// `browser_set_visible(false)` only hides the active tab; this function parks all tab webviews.
 #[tauri::command]
 pub fn browser_hide_all(app: AppHandle) -> Result<(), String> {
-    let ids = {
-        let state = app.state::<AppState>();
-        let tabs = state.tabs.lock().map_err(|_| "lock")?;
-        tabs.order.clone()
-    };
-    for id in ids {
-        let _ = hide_tab(&app, &id);
-    }
+    let ids = { let state = app.state::<AppState>(); let tabs = state.tabs.lock().map_err(|_| "lock")?; tabs.order.clone() };
+    for id in ids { let _ = hide_tab(&app, &id); }
     Ok(())
 }
 
@@ -153,6 +145,8 @@ pub(crate) async fn create_tab_inner(app: &AppHandle, url: &str, _layout: &Viewp
                 if !css.is_empty() { let escaped = css.replace('\\', "\\\\").replace('`', "\\`"); let _ = webview.eval(&format!("(function(){{const s=document.createElement('style');s.id='__muse_shield';s.textContent=`{escaped}`;document.head.appendChild(s)}})();")); }
                 let scriptlet_js = adblock_state.get_injected_script(&url); if !scriptlet_js.is_empty() { let _ = webview.eval(&format!("try{{{scriptlet_js}}}catch(e){{}}")); }
                 let _ = webview.eval("window.__muse_report_favicon && window.__muse_report_favicon()");
+                let title = { let state = app_for_load.state::<AppState>(); state.tabs.lock().ok().and_then(|tabs| tabs.tabs.get(&id_for_load).map(|t| t.title.clone())).unwrap_or_default() };
+                let _ = crate::history::record_visit(&app_for_load, id_for_load.clone(), url.clone(), title);
             }
         })
         .on_document_title_changed(move |_wv, title| { update_tab_field(&app_for_title, &id_for_title, |t| { if !title.trim().is_empty() { t.title = title.clone(); } }); })