firestore_client.ts: Throw FirestoreError instead of Error

Author: dconeybe

packages/firestore/src/core/firestore_client.ts

@@ -232,7 +232,7 @@ export async function setOfflineComponentProvider(
   });
 
   offlineComponentProvider.persistence.setDatabaseDeletedListener(event => {
-    let error: Error | null;
+    let error: FirestoreError | null;
 
     if (event.type === 'ClearSiteDataDatabaseDeletedEvent') {
       const message =
@@ -241,7 +241,9 @@ export async function setOfflineComponentProvider(
         `"Clear Site Data" button in a web browser. ` +
         `Try reloading the web page to re-initialize the ` +
         `IndexedDB database.`;
-      error = new Error(message);
+      // Throw FirestoreError rather than just Error so that the error will
+      // be treated as "non-retryable".
+      error = new FirestoreError('failed-precondition', message);
       logWarn(`${message} event:`, event);
     } else {
       error = null;