watch_change.ts: don't return bloomFilterMightContain if bitCount===0

Author: dconeybe

packages/firestore/src/remote/watch_change.ts

@@ -523,28 +523,19 @@ export class WatchChangeAggregator {
     };
 
     if (bloomFilter.bitCount === 0) {
-      return {
-        status: BloomFilterApplicationStatus.Skipped,
-        bloomFilterMightContain
-      };
+      return { status: BloomFilterApplicationStatus.Skipped };
     }
 
     const removedDocumentCount = this.filterRemovedDocuments(
       watchChange.targetId,
       bloomFilterMightContain
     );
 
-    if (expectedCount !== currentCount - removedDocumentCount) {
-      return {
-        status: BloomFilterApplicationStatus.FalsePositive,
-        bloomFilterMightContain
-      };
-    }
-
-    return {
-      status: BloomFilterApplicationStatus.Success,
-      bloomFilterMightContain
-    };
+    const status =
+      expectedCount === currentCount - removedDocumentCount
+        ? BloomFilterApplicationStatus.Success
+        : BloomFilterApplicationStatus.FalsePositive;
+    return { status, bloomFilterMightContain };
   }
 
   /**