Review/Fix build

Author: schmidt-sebastian

packages/firestore/src/core/event_manager.ts

@@ -46,9 +46,9 @@ export interface Observer<T> {
  * It handles "fan-out". -- Identical queries will re-use the same watch on the
  * backend.
  *
- * PORTING NOTE: On Web, EventManager requires a call to `subscribe()` to
- * register SyncEngine's `listen()` and `unlisten()` functionality. This allows
- * users to tree-shake the Watch logic.
+ * PORTING NOTE: On Web, EventManager `onListen` and `onUnlisten` need to be
+ * assigned to SyncEngine's `listen()` and `unlisten()` API before usage. This
+ * allows users to tree-shake the Watch logic.
  */
 export class EventManager implements SyncEngineListener {
   private queries = new ObjectMap<Query, QueryListenersInfo>(

packages/firestore/src/core/firestore_client.ts

@@ -283,10 +283,8 @@ export class FirestoreClient {
       this.syncEngine = onlineComponentProvider.syncEngine;
       this.eventMgr = onlineComponentProvider.eventManager;
 
-      this.eventMgr.subscribe(
-        syncEngineListen.bind(null, this.syncEngine),
-        syncEngineUnlisten.bind(null, this.syncEngine)
-      );
+      this.eventMgr.onListen = syncEngineListen.bind(null, this.syncEngine);
+      this.eventMgr.onUnlisten = syncEngineUnlisten.bind(null, this.syncEngine);
 
       // When a user calls clearPersistence() in one client, all other clients
       // need to be terminated to allow the delete to succeed.