remove console.log.

Author: ehsannas

packages/firestore/src/local/indexeddb_document_overlay_cache.ts

@@ -104,7 +104,6 @@ export class IndexedDbDocumentOverlayCache implements DocumentOverlayCache {
     largestBatchId: number,
     overlays: MutationMap
   ): PersistencePromise<void> {
-    console.log("in saveOverlays");
     const promises: Array<PersistencePromise<void>> = [];
     overlays.forEach((_, mutation) => {
       const overlay = new Overlay(largestBatchId, mutation);
@@ -213,7 +212,6 @@ export class IndexedDbDocumentOverlayCache implements DocumentOverlayCache {
     transaction: PersistenceTransaction,
     overlay: Overlay
   ): PersistencePromise<void> {
-    console.log("in saveOverlay");
     return documentOverlayStore(transaction).put(
       toDbDocumentOverlay(this.serializer, this.userId, overlay)
     );

packages/firestore/src/local/indexeddb_mutation_queue.ts

@@ -356,7 +356,6 @@ export class IndexedDbMutationQueue implements MutationQueue {
     transaction: PersistenceTransaction,
     documentKeys: SortedMap<DocumentKey, unknown>
   ): PersistencePromise<MutationBatch[]> {
-    console.log("in getAllMutationBatchesAffectingDocumentKeys");
     let uniqueBatchIDs = new SortedSet<BatchId>(primitiveComparator);
 
     const promises: Array<PersistencePromise<void>> = [];
@@ -372,9 +371,6 @@ export class IndexedDbMutationQueue implements MutationQueue {
         (indexKey, _, control) => {
           const [userID, encodedPath, batchID] = indexKey;
 
-          console.log("\t iterating documentMutationStore");
-          console.log(`\t batchId=${batchID}`);
-
           // Only consider rows matching exactly the specific key of
           // interest. Note that because we order by path first, and we
           // order terminators before path separators, we'll encounter all
@@ -395,10 +391,8 @@ export class IndexedDbMutationQueue implements MutationQueue {
       promises.push(promise);
     });
 
-    return PersistencePromise.waitFor(promises).next(() => {
-        console.log(`uniqueBatchIDs.size=${uniqueBatchIDs.size}`);
-        return this.lookupMutationBatches(transaction, uniqueBatchIDs)
-      }
+    return PersistencePromise.waitFor(promises).next(() =>
+      this.lookupMutationBatches(transaction, uniqueBatchIDs)
     );
   }
 
@@ -464,8 +458,6 @@ export class IndexedDbMutationQueue implements MutationQueue {
     transaction: PersistenceTransaction,
     batchIDs: SortedSet<BatchId>
   ): PersistencePromise<MutationBatch[]> {
-    console.log("in lookupMutationBatches");
-    console.log(`\t batchIDs.size=${batchIDs.size}`);
     const results: MutationBatch[] = [];
     const promises: Array<PersistencePromise<void>> = [];
     // TODO(rockwood): Implement this using iterate.
@@ -485,7 +477,6 @@ export class IndexedDbMutationQueue implements MutationQueue {
               mutation.userId === this.userId,
               `Unexpected user '${mutation.userId}' for mutation batch ${batchId}`
             );
-            console.log(`\t found mutation in mutationStore`);
             results.push(fromDbMutationBatch(this.serializer, mutation));
           })
       );

packages/firestore/src/local/indexeddb_persistence.ts

@@ -217,7 +217,6 @@ export class IndexedDbPersistence implements Persistence {
     private readonly forceOwningTab: boolean,
     private readonly schemaVersion = SCHEMA_VERSION
   ) {
-    console.log(`SCHEMA_VERSION=${schemaVersion}`);
     if (!IndexedDbPersistence.isAvailable()) {
       throw new FirestoreError(
         Code.UNIMPLEMENTED,

packages/firestore/src/local/indexeddb_schema_converter.ts

@@ -143,7 +143,6 @@ export class SchemaConverter implements SimpleDbSchemaConverter {
     fromVersion: number,
     toVersion: number
   ): PersistencePromise<void> {
-    console.log(`in createOrUpgrade from ${fromVersion} to ${toVersion}`);
     debugAssert(
       fromVersion < toVersion &&
         fromVersion >= 0 &&
@@ -250,7 +249,6 @@ export class SchemaConverter implements SimpleDbSchemaConverter {
     }
 
     if (fromVersion < 14 && toVersion >= 14) {
-      console.log("about to runOverlayMigration");
       p = p.next(() => this.runOverlayMigration(db, simpleDbTransaction));
     }
 
@@ -473,7 +471,6 @@ export class SchemaConverter implements SimpleDbSchemaConverter {
     db: IDBDatabase,
     transaction: SimpleDbTransaction
   ): PersistencePromise<void> {
-    console.log("in runOverlayMigration");
     const mutationsStore = transaction.store<
       DbMutationBatchKey,
       DbMutationBatch

packages/firestore/src/local/local_documents_view.ts

@@ -215,7 +215,6 @@ export class LocalDocumentsView {
     transaction: PersistenceTransaction,
     docs: MutableDocumentMap
   ): PersistencePromise<void> {
-    console.log("in recalculateAndSaveOverlays")
     const masks = newDocumentKeyMap<FieldMask | null>();
     // A reverse lookup map from batch id to the documents within that batch.
     let documentsByBatchId = new SortedMap<number, DocumentKeySet>(
@@ -225,13 +224,9 @@ export class LocalDocumentsView {
     return this.mutationQueue
       .getAllMutationBatchesAffectingDocumentKeys(transaction, docs)
       .next(batches => {
-        console.log("back in recalculateAndSaveOverlays");
-        console.log(`got ${batches.length} batches`);
         for (const batch of batches) {
           batch.keys().forEach(key => {
-            console.log(`\t key=${key.toString()}`);
             const baseDoc = docs.get(key);
-            console.log(`\t baseDoc=${baseDoc ? baseDoc.toString() : "null"}`);
             if (baseDoc === null) {
               return;
             }
@@ -249,29 +244,21 @@ export class LocalDocumentsView {
         }
       })
       .next(() => {
-        console.log("documentsByBatchId=");
-        console.log(documentsByBatchId);
-        console.log("masks=");
-        console.log(masks);
         const promises: Array<PersistencePromise<void>> = [];
         // Iterate in descending order of batch IDs, and skip documents that are
         // already saved.
         const iter = documentsByBatchId.getReverseIterator();
         while (iter.hasNext()) {
-          console.log(`reverse iteration over documentsByBatchId`);
           const entry = iter.getNext();
           const batchId = entry.key;
           const keys = entry.value;
-          console.log(`\t batchId=${batchId}`);
-          console.log(`\t keys=${keys}`);
           const overlays = newMutationMap();
           keys.forEach(key => {
             if (!processed.has(key)) {
               const overlayMutation = calculateOverlayMutation(
                 docs.get(key)!,
                 masks.get(key)!
               );
-              console.log(`calculateOverlayMigration resulted in ${overlayMutation ? overlayMutation?.type.toString() : "null"}`);
               if (overlayMutation !== null) {
                 overlays.set(key, overlayMutation);
               }
@@ -298,7 +285,6 @@ export class LocalDocumentsView {
     transaction: PersistenceTransaction,
     documentKeys: DocumentKeySet
   ): PersistencePromise<void> {
-    console.log("in recalculateAndSaveOverlaysForDocumentKeys");
     return this.remoteDocumentCache
       .getEntries(transaction, documentKeys)
       .next(docs => this.recalculateAndSaveOverlays(transaction, docs));