Skip to content

Commit

Permalink
Sends beacon request in beforeunload event in Firefox
Browse files Browse the repository at this point in the history
Firefox has a bug/feature that it don't send the beacon in pagehide event.

Closes #19305
  • Loading branch information
mstahv committed Oct 9, 2024
1 parent 1fbf412 commit e4af719
Showing 1 changed file with 10 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,16 @@ public void start(ValueMap initialUidl) {
registry.getMessageHandler().handleMessage(initialUidl);
}

Browser.getWindow().addEventListener("pagehide", e -> {
registry.getMessageSender().sendUnloadBeacon();
});
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("pageshow", e -> {
// Currently only Safari gets here, sometimes when going back/foward
Expand Down

0 comments on commit e4af719

Please sign in to comment.