Fix isWindowSupported() check (#5957)

Author: hsubox76

.changeset/tame-apes-tie.md

@@ -0,0 +1,5 @@
+---
+'@firebase/messaging': patch
+---
+
+Fix uncaught rejection in `isSupported()` if environment does not support IndexedDB's `open()` method.

packages/messaging-compat/src/messaging-compat.ts

@@ -65,6 +65,8 @@ export function isSupported(): boolean {
 
 /**
  * Checks to see if the required APIs exist.
+ * Unlike the modular version, it does not check if IndexedDB.open() is allowed
+ * in order to keep isSupported() synchronous and maintain v8 compatibility.
  */
 function isWindowSupported(): boolean {
   return (

packages/messaging/src/api/isSupported.ts

@@ -28,13 +28,19 @@ import {
  * @public
  */
 export async function isWindowSupported(): Promise<boolean> {
+  try {
+    // This throws if open() is unsupported, so adding it to the conditional
+    // statement below can cause an uncaught error.
+    await validateIndexedDBOpenable();
+  } catch (e) {
+    return false;
+  }
   // firebase-js-sdk/issues/2393 reveals that idb#open in Safari iframe and Firefox private browsing
   // might be prohibited to run. In these contexts, an error would be thrown during the messaging
   // instantiating phase, informing the developers to import/call isSupported for special handling.
   return (
     typeof window !== 'undefined' &&
     isIndexedDBAvailable() &&
-    (await validateIndexedDBOpenable()) &&
     areCookiesEnabled() &&
     'serviceWorker' in navigator &&
     'PushManager' in window &&