Merge ee12bb1 into 2c90aec

firebase · Aug 17, 2021 · f056cd1 · f056cd1
2 parents 2c90aec + ee12bb1
commit f056cd1
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/.changeset/witty-chicken-mate.md b/.changeset/witty-chicken-mate.md
@@ -0,0 +1,5 @@
+---
+"@firebase/firestore": patch
+---
+
+Fixed a regression that prevented the garbage collector from running if multi-tab was disabled.
diff --git a/packages/firestore/src/core/component_provider.ts b/packages/firestore/src/core/component_provider.ts
@@ -182,6 +182,15 @@ export class IndexedDbOfflineComponentProvider extends MemoryOfflineComponentPro
       this.onlineComponentProvider.syncEngine
     );
     await fillWritePipeline(this.onlineComponentProvider.remoteStore);
+
+    // NOTE: This will immediately call the listener, so we make sure to
+    // set it after localStore / remoteStore are started.
+    await this.persistence.setPrimaryStateListener(() => {
+      if (this.gcScheduler && !this.gcScheduler.started) {
+          this.gcScheduler.start(this.localStore);
+      }
+      return Promise.resolve();
+    });
   }
 
   createLocalStore(cfg: ComponentConfiguration): LocalStore {