Set FirebaseError names to be calling constructors name

common/api-review/util.api.md

@@ -213,7 +213,6 @@ export class FirebaseError extends Error {
     customData?: Record<string, unknown> | undefined);
     readonly code: string;
     customData?: Record<string, unknown> | undefined;
-    readonly name: string;
 }
 
 // Warning: (ae-missing-release-tag) "FirebaseSignInProvider" is exported by the package, but it is missing a release tag (@alpha, @beta, @public, or @internal)

packages/util/src/errors.ts

@@ -72,9 +72,6 @@ export interface ErrorData {
 // Based on code from:
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error#Custom_Error_Types
 export class FirebaseError extends Error {
-  /** The custom name for all FirebaseErrors. */
-  readonly name: string = ERROR_NAME;
-
   constructor(
     /** The error code for this error. */
     readonly code: string,
@@ -84,6 +81,9 @@ export class FirebaseError extends Error {
   ) {
     super(message);
 
+    // Set the name of the error to the name of the calling constructor.
+    this.name = this.constructor.name;
+
     // Fix For ES5
     // https://github.com/Microsoft/TypeScript-wiki/blob/master/Breaking-Changes.md#extending-built-ins-like-error-array-and-map-may-no-longer-work
     // TODO(dlarocque): Replace this with `new.target`: https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-2.html#support-for-newtarget