Even more feedbacks.

Author: wu-hui

packages/firestore/src/core/bundle.ts

@@ -143,7 +143,7 @@ export function bundleSuccessProgress(
 export class BundleLoadResult {
   constructor(
     readonly progress: firestore.LoadBundleTaskProgress,
-    readonly changedDocs?: MaybeDocumentMap
+    readonly changedDocs: MaybeDocumentMap
   ) {}
 }
 

packages/firestore/src/core/sync_engine.ts

@@ -1432,15 +1432,13 @@ async function loadBundleImpl(
     }
 
     const result = await loader.complete();
-    if (result.changedDocs) {
-      // TODO(b/160876443): This currently raises snapshots with
-      // `fromCache=false` if users already listen to some queries and bundles
-      // has newer version.
-      await syncEngine.emitNewSnapsAndNotifyLocalStore(
-        result.changedDocs,
-        /* remoteEvent */ undefined
-      );
-    }
+    // TODO(b/160876443): This currently raises snapshots with
+    // `fromCache=false` if users already listen to some queries and bundles
+    // has newer version.
+    await syncEngine.emitNewSnapsAndNotifyLocalStore(
+      result.changedDocs,
+      /* remoteEvent */ undefined
+    );
 
     // Save metadata, so loading the same bundle will skip.
     await saveBundle(syncEngine.localStore, metadata);