Issue 2393 - Add environment check to Performance Module

.changeset/little-cycles-fold.md

@@ -0,0 +1,6 @@
+---
+"firebase": minor
+"@firebase/performance": minor
+---
+
+Issue 2393 - Add environment check to Performance Module

packages/performance/index.ts

@@ -26,7 +26,6 @@ import { ERROR_FACTORY, ErrorCode } from './src/utils/errors';
 import { FirebasePerformance } from '@firebase/performance-types';
 import { Component, ComponentType } from '@firebase/component';
 import { FirebaseInstallations } from '@firebase/installations-types';
-
 import { name, version } from './package.json';
 
 const DEFAULT_ENTRY_NAME = '[DEFAULT]';

packages/performance/src/controllers/perf.ts

@@ -21,19 +21,18 @@ import { getInitializationPromise } from '../services/initialization_service';
 import { Api } from '../services/api_service';
 import { FirebaseApp } from '@firebase/app-types';
 import { FirebasePerformance } from '@firebase/performance-types';
-import { consoleLogger } from '../utils/console_logger';
 import { setupTransportService } from '../services/transport_service';
-
 export class PerformanceController implements FirebasePerformance {
   constructor(readonly app: FirebaseApp) {
-    if (Api.getInstance().requiredApisAvailable()) {
-      setupTransportService();
-      getInitializationPromise().then(setupOobResources, setupOobResources);
-    } else {
-      consoleLogger.info(
-        'Firebase Performance cannot start if browser does not support fetch and Promise or cookie is disabled.'
-      );
-    }
+    // eslint-disable-next-line @typescript-eslint/no-floating-promises
+    Api.getInstance()
+      .requiredApisAvailable()
+      .then(isAvailable => {
+        if (isAvailable) {
+          setupTransportService();
+          getInitializationPromise().then(setupOobResources, setupOobResources);
+        }
+      });
   }
 
   trace(name: string): Trace {

packages/performance/src/services/api_service.ts

@@ -16,7 +16,11 @@
  */
 
 import { ERROR_FACTORY, ErrorCode } from '../utils/errors';
-
+import {
+  isIndexedDBAvailable,
+  validateIndexedDBOpenable
+} from '@firebase/util';
+import { consoleLogger } from '../utils/console_logger';
 declare global {
   interface Window {
     PerformanceObserver: typeof PerformanceObserver;
@@ -109,11 +113,30 @@ export class Api {
     );
   }
 
-  requiredApisAvailable(): boolean {
-    if (fetch && Promise && this.navigator && this.navigator.cookieEnabled) {
+  async requiredApisAvailable(): Promise<boolean> {
+    if (
+      !fetch ||
+      !Promise ||
+      !this.navigator ||
+      !this.navigator.cookieEnabled
+    ) {
+      consoleLogger.info(
+        'Firebase Performance cannot start if browser does not support fetch and Promise or cookie is disabled.'
+      );
+      return false;
+    }
+
+    if (!isIndexedDBAvailable()) {
+      consoleLogger.info('IndexedDB is not supported by current browswer');
+      return false;
+    }
+    try {
+      await validateIndexedDBOpenable();
       return true;
+    } catch (error) {
+      consoleLogger.info(`Environment doesn't support IndexedDB: ${error}`);
+      return false;
     }
-    return false;
   }
 
   setupObserver(