issue firebase#2393 fix for analytics module

Author: XuechunHou

packages/analytics/index.ts

@@ -56,6 +56,10 @@ export function registerAnalytics(instance: _FirebaseNamespace): void {
           .getProvider('installations')
           .getImmediate();
 
+        if (!isSupported()) {
+          throw ERROR_FACTORY.create(AnalyticsError.UNSUPPORTED_BROWSER);
+        }
+
         return factory(app, installations);
       },
       ComponentType.PUBLIC
@@ -97,8 +101,50 @@ registerAnalytics(firebase as _FirebaseNamespace);
 declare module '@firebase/app-types' {
   interface FirebaseNamespace {
     analytics(app?: FirebaseApp): FirebaseAnalytics;
+    isSupported(): boolean;
   }
   interface FirebaseApp {
     analytics(): FirebaseAnalytics;
   }
 }
+
+function isSupported(): boolean {
+  if (self && 'ServiceWorkerGlobalScope' in self) {
+    // Running in ServiceWorker context
+    return isSWControllerSupported();
+  } else {
+    // Assume we are in the window context.
+    return isWindowControllerSupported();
+  }
+}
+
+/**
+ * Checks to see if the required APIs exist.
+ */
+function isWindowControllerSupported(): boolean {
+  return (
+    'indexedDB' in window &&
+    indexedDB !== null &&
+    navigator.cookieEnabled &&
+    'serviceWorker' in navigator &&
+    'PushManager' in window &&
+    'Notification' in window &&
+    'fetch' in window &&
+    ServiceWorkerRegistration.prototype.hasOwnProperty('showNotification') &&
+    PushSubscription.prototype.hasOwnProperty('getKey')
+  );
+}
+
+/**
+ * Checks to see if the required APIs exist within SW Context.
+ */
+function isSWControllerSupported(): boolean {
+  return (
+    'indexedDB' in self &&
+    indexedDB !== null &&
+    'PushManager' in self &&
+    'Notification' in self &&
+    ServiceWorkerRegistration.prototype.hasOwnProperty('showNotification') &&
+    PushSubscription.prototype.hasOwnProperty('getKey')
+  );
+}

packages/analytics/src/errors.ts

@@ -22,7 +22,8 @@ export const enum AnalyticsError {
   NO_GA_ID = 'no-ga-id',
   ALREADY_EXISTS = 'already-exists',
   ALREADY_INITIALIZED = 'already-initialized',
-  INTEROP_COMPONENT_REG_FAILED = 'interop-component-reg-failed'
+  INTEROP_COMPONENT_REG_FAILED = 'interop-component-reg-failed',
+  UNSUPPORTED_BROWSER = 'unsupported-browser'
 }
 
 const ERRORS: ErrorMap<AnalyticsError> = {
@@ -39,7 +40,9 @@ const ERRORS: ErrorMap<AnalyticsError> = {
     'settings() must be called before initializing any Analytics instance' +
     'or it will have no effect.',
   [AnalyticsError.INTEROP_COMPONENT_REG_FAILED]:
-    'Firebase Analytics Interop Component failed to instantiate'
+    'Firebase Analytics Interop Component failed to instantiate',
+  [AnalyticsError.UNSUPPORTED_BROWSER]:
+    "This browser doesn't support the API's required to use the firebase SDK."
 };
 
 interface ErrorParams {

packages/firebase/index.d.ts

@@ -5083,6 +5083,7 @@ declare namespace firebase.analytics {
     id?: string;
     name?: string;
   }
+  function isSupported(): boolean;
 }
 
 declare namespace firebase.auth.Auth {