'pagehide' all the way

Author: wu-hui

packages/firestore/src/local/indexeddb_persistence.ts

@@ -23,7 +23,7 @@ import { debugAssert } from '../util/assert';
 import { AsyncQueue, DelayedOperation, TimerId } from '../util/async_queue';
 import { Code, FirestoreError } from '../util/error';
 import { logDebug, logError } from '../util/log';
-import { DocumentLike, WindowLike, terminationEvent } from '../util/types';
+import { DocumentLike, WindowLike } from '../util/types';
 
 import { BundleCache } from './bundle_cache';
 import { IndexManager } from './index_manager';
@@ -966,10 +966,7 @@ export class IndexedDbPersistence implements Persistence {
           return this.shutdown();
         });
       };
-      this.window.addEventListener(
-        terminationEvent(this.window),
-        this.windowUnloadHandler
-      );
+      this.window.addEventListener('pagehide', this.windowUnloadHandler);
     }
   }
 
@@ -979,10 +976,7 @@ export class IndexedDbPersistence implements Persistence {
         typeof this.window?.removeEventListener === 'function',
         "Expected 'window.removeEventListener' to be a function"
       );
-      this.window!.removeEventListener(
-        terminationEvent(this.window),
-        this.windowUnloadHandler
-      );
+      this.window!.removeEventListener('pagehide', this.windowUnloadHandler);
       this.windowUnloadHandler = null;
     }
   }

packages/firestore/src/local/shared_client_state.ts

@@ -32,7 +32,7 @@ import { logError, logDebug } from '../util/log';
 import { primitiveComparator } from '../util/misc';
 import { SortedMap } from '../util/sorted_map';
 import { SortedSet } from '../util/sorted_set';
-import { isSafeInteger, terminationEvent, WindowLike } from '../util/types';
+import { isSafeInteger, WindowLike } from '../util/types';
 
 import {
   CLIENT_STATE_KEY_PREFIX,
@@ -613,9 +613,7 @@ export class WebStorageSharedClientState implements SharedClientState {
 
     // Register a window unload hook to remove the client metadata entry from
     // WebStorage even if `shutdown()` was not called.
-    this.window.addEventListener(terminationEvent(this.window), () =>
-      this.shutdown()
-    );
+    this.window.addEventListener('pagehide', () => this.shutdown());
 
     this.started = true;
   }

packages/firestore/src/util/types.ts

@@ -59,14 +59,6 @@ export interface WindowLike {
   removeEventListener(type: string, listener: EventListener): void;
 }
 
-/**
- * Returns the page termination event to listen to. 'pagehide' is recommended
- * if available, it falls back to the less reliable 'unload'.
- */
-export function terminationEvent(window: WindowLike | null): string {
-  return 'onpagehide' in window! ? 'pagehide' : 'unload';
-}
-
 /** The subset of the browser's Document interface used by the SDK. */
 export interface DocumentLike {
   readonly visibilityState: VisibilityState;