Remove Promise from FirebaseNamespace (#1741)

* Remove Promsie from FirebaseNamespae

* remove the test case that checks promise

Author: Feiyang1

integration/shared/namespaceDefinition.json

@@ -38,26 +38,6 @@
   "apps": {
     "__type": "array"
   },
-  "Promise": {
-    "__type": "function",
-    "resolve": {
-      "__type": "function"
-    },
-    "reject": {
-      "__type": "function"
-    },
-    "all": {
-      "__type": "function"
-    },
-    "prototype": {
-      "then": {
-        "__type": "function"
-      },
-      "catch": {
-        "__type": "function"
-      }
-    }
-  },
   "INTERNAL": {
     "__type": "object",
     "registerService": {

packages/app-types/index.d.ts

@@ -98,10 +98,6 @@ export interface FirebaseNamespace {
    */
   apps: FirebaseApp[];
 
-  // Inherit the type information of our exported Promise implementation from
-  // es6-promises.
-  Promise: typeof Promise;
-
   // The current SDK version.
   SDK_VERSION: string;
 }

packages/app/src/firebaseNamespaceCore.ts

@@ -62,7 +62,6 @@ export function createFirebaseNamespaceCore(
     initializeApp: initializeApp,
     app: app as any,
     apps: null as any,
-    Promise: Promise,
     SDK_VERSION: '${JSCORE_VERSION}',
     INTERNAL: {
       registerService,
@@ -244,7 +243,6 @@ export function createFirebaseNamespaceCore(
     }
 
     const useService = name;
-    const options = app.options;
 
     return useService;
   }