Rename

Author: schmidt-sebastian

packages/firestore/src/core/sync_engine.ts

@@ -860,7 +860,7 @@ export class SyncEngine implements RemoteSyncer {
     );
   }
 
-  async handleUserChange(user: User): Promise<void> {
+  async handleCredentialChange(user: User): Promise<void> {
     const userChanged = !this.currentUser.isEqual(user);
 
     if (userChanged) {

packages/firestore/src/remote/remote_store.ts

@@ -776,7 +776,7 @@ export class RemoteStore implements TargetMetadataProvider {
 
     await this.disableNetworkInternal();
     this.onlineStateTracker.set(OnlineState.Unknown);
-    await this.syncEngine.handleUserChange(user);
+    await this.syncEngine.handleCredentialChange(user);
 
     this.offlineCauses.delete(OfflineCause.CredentialChange);
     await this.enableNetworkInternal();

packages/firestore/src/remote/remote_syncer.ts

@@ -71,5 +71,5 @@ export interface RemoteSyncer {
    * Updates all local state to match the pending mutations for the given user.
    * May be called repeatedly for the same user.
    */
-  handleUserChange(user: User): Promise<void>;
+  handleCredentialChange(user: User): Promise<void>;
 }

packages/firestore/test/unit/specs/spec_test_runner.ts

@@ -716,7 +716,7 @@ abstract class TestRunner {
 
   private async doChangeUser(user: string | null): Promise<void> {
     this.user = new User(user);
-    // We don't block on `handleCredentialChange` as it may not get executed
+    // We don't block on `handleUserChange` as it may not get executed
     // during an IndexedDb failure. Non-recovery tests will pick up the user
     // change when the AsyncQueue is drained.
     this.queue.enqueueRetryable(() =>