Update Feat/cef with upstream

[senamakel]

Summary by CodeRabbit

• New Features

  • NSIS uninstaller icon/header image; Android/iOS file associations; iOS flags (--no-sign, --archive-only)
  • WebView eval-with-callback and generalAutofill toggle
  • Mobile: Suspended/Resumed events, activity/scene getters, supportsMultipleWindows API
  • Tray: setIconWithAsTemplate (macOS); "Bring All to Front" menu item

• Bug Fixes

  • WebView2 detection/installer flow, NSIS bundling fixes, iOS Swift library search path, macOS signing base64 decoding fix

• Enhancements

  • Android debugApplicationIdSuffix, emulator restart prompt, Android manifest/Gradle improvements, various tooling and schema updates

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@crates/tauri/src/manager/webview.rs`:
- Around line 308-329: The current code replaces the
pending.on_web_content_process_terminate_handler with only the app-level hook,
removing the runtime fallback (Builder::on_web_content_process_terminate)
behavior; instead, compose a new handler that invokes both: capture the existing
runtime/built-in handler and the
app_manager_.webview.on_web_content_process_terminate hook and replace
pending.on_web_content_process_terminate_handler with a closure that, when
fired, looks up the webview via manager.manager_owned().get_webview(&label_) and
then calls the app hook (on_web_content_process_terminate) and also invokes the
existing runtime fallback handler so the default reload/recovery still
runs—preserve call order and ensure both handlers run safely if either is None.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 30c4e72f-ce6a-41aa-938d-e0f9c67bdb3e

📥 Commits

Reviewing files that changed from the base of the PR and between 5054a1c and 7f01af6.

📒 Files selected for processing (3)

• crates/tauri-runtime-cef/src/cef_impl.rs
• crates/tauri/scripts/core.js
• crates/tauri/src/manager/webview.rs

✅ Files skipped from review due to trivial changes (1)

• crates/tauri-runtime-cef/src/cef_impl.rs

[coderabbitai]

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Compose the app hook instead of replacing the runtime fallback.

Line 309 makes the app-level hook a fallback-only path, and once this closure is installed wry no longer uses its built-in reload-on-termination behavior. That means Builder::on_web_content_process_terminate is skipped for webviews that already set their own handler, and all other webviews lose the default recovery path after a renderer crash.

Suggested direction

     #[cfg(any(target_os = "macos", target_os = "ios"))]
-    if pending.on_web_content_process_terminate_handler.is_none() {
-      let app_manager_ = manager.manager_owned();
-      if app_manager_
-        .webview
-        .on_web_content_process_terminate
-        .is_some()
-      {
-        let label_ = pending.label.clone();
-        pending
-          .on_web_content_process_terminate_handler
-          .replace(Box::new(move || {
-            if let Some(w) = app_manager_.get_webview(&label_)
-              && let Some(on_web_content_process_terminate) =
-                &app_manager_.webview.on_web_content_process_terminate
-            {
-              on_web_content_process_terminate(&w);
-            }
-          }));
-      }
-    }
+    if let Some(app_hook) = app_manager
+      .webview
+      .on_web_content_process_terminate
+      .clone()
+    {
+      let app_manager_ = manager.manager_owned();
+      let label_ = pending.label.clone();
+      let webview_hook = pending.on_web_content_process_terminate_handler.take();
+      pending
+        .on_web_content_process_terminate_handler
+        .replace(Box::new(move || {
+          if let Some(w) = app_manager_.get_webview(&label_) {
+            app_hook(&w);
+            if let Some(handler) = &webview_hook {
+              handler();
+            } else {
+              let _ = w.reload();
+            }
+          }
+        }));
+    }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@crates/tauri/src/manager/webview.rs` around lines 308 - 329, The current code
replaces the pending.on_web_content_process_terminate_handler with only the
app-level hook, removing the runtime fallback
(Builder::on_web_content_process_terminate) behavior; instead, compose a new
handler that invokes both: capture the existing runtime/built-in handler and the
app_manager_.webview.on_web_content_process_terminate hook and replace
pending.on_web_content_process_terminate_handler with a closure that, when
fired, looks up the webview via manager.manager_owned().get_webview(&label_) and
then calls the app hook (on_web_content_process_terminate) and also invokes the
existing runtime fallback handler so the default reload/recovery still
runs—preserve call order and ensure both handlers run safely if either is None.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@crates/tauri/src/webview/mod.rs`:
- Around line 84-90: The origin check in is_url_for_custom_protocol currently
compares only scheme and domain, causing mismatches when ports differ; update
is_url_for_custom_protocol to compare scheme, host_str() and
port_or_known_default() (use Url::host_str() and Url::port_or_known_default())
so origin includes port per RFC6454, and apply the same change to the tauri
branch of is_local_url; also add regression tests asserting that
https://tauri.localhost:8443/ and https://myproto.localhost:8443/ are treated as
distinct origins from the default-port variants.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f3589f53-ce67-4178-8a97-d5868ef10b86

📥 Commits

Reviewing files that changed from the base of the PR and between 7f01af6 and d035242.

⛔ Files ignored due to path filters (1)

• Cargo.lock is excluded by !**/*.lock

📒 Files selected for processing (15)

• .changes/driver-arg-eq-separator.md
• .changes/enforce-acl-remote-origins.md
• .changes/expose-mobile-monitor-apis.md
• .changes/fix-request-permission-android.md
• .changes/phf-0.13.md
• .changes/tauri-sec-localhost-suffix.md
• crates/tauri-cli/Cargo.toml
• crates/tauri-driver/Cargo.toml
• crates/tauri-runtime-cef/src/cef_impl.rs
• crates/tauri-utils/Cargo.toml
• crates/tauri/mobile/android-codegen/TauriActivity.kt
• crates/tauri/src/test/mod.rs
• crates/tauri/src/webview/mod.rs
• crates/tauri/src/window/plugin.rs
• examples/api/src-tauri/src/lib.rs

✅ Files skipped from review due to trivial changes (10)

• .changes/phf-0.13.md
• .changes/driver-arg-eq-separator.md
• crates/tauri-driver/Cargo.toml
• .changes/tauri-sec-localhost-suffix.md
• .changes/expose-mobile-monitor-apis.md
• .changes/enforce-acl-remote-origins.md
• crates/tauri/src/test/mod.rs
• .changes/fix-request-permission-android.md
• crates/tauri/mobile/android-codegen/TauriActivity.kt
• crates/tauri-runtime-cef/src/cef_impl.rs

🚧 Files skipped from review as they are similar to previous changes (3)

• crates/tauri-cli/Cargo.toml
• crates/tauri-utils/Cargo.toml
• crates/tauri/src/window/plugin.rs