Fix reported freeze

CHANGELOG.md

@@ -12,6 +12,7 @@
 
 - Silent failure when opening Flutter projects without `.idea` directory in IntelliJ IDEA, by removing `FlutterProjectOpenProcessor` and migrating configuration logic to `FlutterInitializer`. (#8845)
 - Gutter buttons not running tests with non-ASCII characters in their names. (#7985)
+- Freeze from JX Browser close. (#8864)
 
 ## 90.0.0
 

src/io/flutter/jxbrowser/EmbeddedBrowserEngine.java

@@ -5,10 +5,11 @@
  */
 package io.flutter.jxbrowser;
 
-import com.intellij.openapi.application.ApplicationListener;
+import com.intellij.openapi.Disposable;
 import com.intellij.openapi.application.ApplicationManager;
 import com.intellij.openapi.diagnostic.Logger;
 import com.intellij.openapi.util.SystemInfo;
+import io.flutter.utils.OpenApiUtils;
 import com.teamdev.jxbrowser.engine.Engine;
 import com.teamdev.jxbrowser.engine.EngineOptions;
 import com.teamdev.jxbrowser.engine.PasswordStore;
@@ -22,7 +23,7 @@
 import static com.teamdev.jxbrowser.engine.RenderingMode.HARDWARE_ACCELERATED;
 import static com.teamdev.jxbrowser.engine.RenderingMode.OFF_SCREEN;
 
-public class EmbeddedBrowserEngine {
+public class EmbeddedBrowserEngine implements Disposable {
   private static final @NotNull Logger LOG = PluginLogger.createLogger(EmbeddedBrowserEngine.class);
   private final Engine engine;
 
@@ -60,22 +61,22 @@ public EmbeddedBrowserEngine() {
     }
     engine = temp;
 
-    ApplicationManager.getApplication().addApplicationListener(new ApplicationListener() {
-      @Override
-      public boolean canExitApplication() {
-        try {
-          if (engine != null && !engine.isClosed()) {
-            engine.close();
-          }
+  }
+
+  @Override
+  public void dispose() {
+    OpenApiUtils.safeExecuteOnPooledThread(() -> {
+      try {
+        if (engine != null && !engine.isClosed()) {
+          engine.close();
         }
-        catch (Exception ex) {
-          if (FlutterSettings.getInstance().isFilePathLoggingEnabled()) {
-            LOG.info(ex);
-          } else {
-            LOG.info("Exception when closing JX Browser engine: " + ex.getMessage());
-          }
+      }
+      catch (Exception ex) {
+        if (FlutterSettings.getInstance().isFilePathLoggingEnabled()) {
+          LOG.info(ex);
+        } else {
+          LOG.info("Exception when closing JX Browser engine: " + ex.getMessage());
         }
-        return true;
       }
     });
   }

src/io/flutter/jxbrowser/EmbeddedJxBrowser.java

@@ -28,6 +28,7 @@
 import io.flutter.settings.FlutterSettings;
 import io.flutter.utils.AsyncUtils;
 import io.flutter.utils.JxBrowserUtils;
+import io.flutter.utils.OpenApiUtils;
 import io.flutter.utils.ZoomLevelSelector;
 import io.flutter.view.EmbeddedBrowser;
 import io.flutter.view.EmbeddedTab;
@@ -98,7 +99,18 @@ public void loadUrl(String url) {
 
   @Override
   public void close() {
-    this.browser.close();
+    OpenApiUtils.safeExecuteOnPooledThread(() -> {
+      try {
+        this.browser.close();
+      }
+      catch (Exception ex) {
+        if (FlutterSettings.getInstance().isFilePathLoggingEnabled()) {
+          LOG.info(ex);
+        } else {
+          LOG.info("Exception when closing JX Browser instance: " + ex.getMessage());
+        }
+      }
+    });
   }
 
   @Override

src/io/flutter/view/EmbeddedBrowser.java

@@ -69,14 +69,15 @@ public void projectClosing(@NotNull Project project) {
           final Map<String, BrowserTab> tabs = windows.get(window);
           for (final String tabName : tabs.keySet()) {
             final BrowserTab tab = tabs.get(tabName);
-            if (tab.embeddedTab != null) {
-              try {
-                tab.embeddedTab.close();
+              final EmbeddedTab embeddedTab = tab.embeddedTab;
+              if (embeddedTab != null) {
+                try {
+                  embeddedTab.close();
+                }
+                catch (Exception ex) {
+                  logger().info(ex);
+                }
               }
-              catch (Exception ex) {
-                logger().info(ex);
-              }
-            }
           }
           tabs.clear();
         }

src/io/flutter/widgetpreview/WidgetPreviewPanel.java

@@ -264,7 +264,12 @@ public void dispose() {
     // Dispose the browser tab
     final EmbeddedTab tab = browserTabRef.getAndSet(null);
     if (tab != null) {
-      tab.close();
+      try {
+        tab.close();
+      }
+      catch (Exception ex) {
+        LOG.info(ex);
+      }
     }
   }
 }