From 07aaa4d842fdecaf811585050d1dd5b04d5ef59a Mon Sep 17 00:00:00 2001 From: Vaadin Bot Date: Fri, 11 Oct 2024 09:04:33 +0200 Subject: [PATCH] Revert "Sends beacon request in beforeunload event in Firefox (#20198)" (#20225) (#20227) This reverts commit e554b456891e6cdc112fda9e02872b4d6de639a9. Co-authored-by: Teppo Kurki --- .../com/vaadin/client/ApplicationConnection.java | 13 +++---------- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/flow-client/src/main/java/com/vaadin/client/ApplicationConnection.java b/flow-client/src/main/java/com/vaadin/client/ApplicationConnection.java index 62fab73ff9b..1f3db48ce8c 100644 --- a/flow-client/src/main/java/com/vaadin/client/ApplicationConnection.java +++ b/flow-client/src/main/java/com/vaadin/client/ApplicationConnection.java @@ -116,16 +116,9 @@ public void start(ValueMap initialUidl) { registry.getMessageHandler().handleMessage(initialUidl); } - if (BrowserInfo.get().isFirefox()) { - // Sends in beforeunload in FF (don't support beacon in pagehide) - Browser.getWindow().addEventListener("beforeunload", e -> { - registry.getMessageSender().sendUnloadBeacon(); - }); - } else { - Browser.getWindow().addEventListener("pagehide", e -> { - registry.getMessageSender().sendUnloadBeacon(); - }); - } + Browser.getWindow().addEventListener("pagehide", e -> { + registry.getMessageSender().sendUnloadBeacon(); + }); Browser.getWindow().addEventListener("pageshow", e -> { // Currently only Safari gets here, sometimes when going back/foward