Add Firefox Beacon bug workaround

TatuLund · Jul 3, 2024 · a29f618 · a29f618
1 parent fadabed
commit a29f618
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/vaadincreate-ui/src/main/java/org/vaadin/tatu/vaadincreate/VaadinCreateUI.java b/vaadincreate-ui/src/main/java/org/vaadin/tatu/vaadincreate/VaadinCreateUI.java
@@ -60,6 +60,7 @@ public class VaadinCreateUI extends UI implements EventBusListener, HasI18N {
     @Override
     protected void init(VaadinRequest request) {
         getPage().setTitle("Vaadin Create 23'");
+        addFirefoxBrowserCloseWorkaround();
         if (!getAccessControl().isUserSignedIn()) {
             showLoginView();
         } else {
@@ -69,6 +70,21 @@ protected void init(VaadinRequest request) {
         eventBus.registerEventBusListener(this);
     }
 
+    // Vaadin 8 actually has page hide listener and eager closing of UI's, but
+    // there is a bug in Firefox's sending Beacon upon browser close, hence
+    // adding a workaround to detect closing of the Browser old fashioned way.
+    // https://bugzilla.mozilla.org/show_bug.cgi?id=1609653
+    private void addFirefoxBrowserCloseWorkaround() {
+        if (getPage().getWebBrowser().isFirefox()) {
+            getPage().getJavaScript()
+                    .execute("function closeListener() { catchClose(); } "
+                            + "window.addEventListener('beforeunload', closeListener);");
+            getPage().getJavaScript().addFunction("catchClose", arguments -> {
+                close();
+            });
+        }
+    }
+
     private void showLoginView() {
         setContent(new LoginView(getAccessControl(), e -> onLogin()));
     }