resetState is actually only used internally

Author: hughns

src/matrixrtc/NewMembershipManager.ts

@@ -168,7 +168,6 @@ export class MembershipManager implements IMembershipManager {
         this.focusActive = focusActive;
         this.leavePromiseDefer = undefined;
         if (!this.scheduler.state.running) {
-            this.scheduler.resetState();
             this.scheduler.state.running = true;
             this.scheduler.startWithJoin().catch((e) => {
                 // Set the rtc session to left state since we cannot recover from here and the consumer user of the

src/matrixrtc/NewMembershipManagerActionScheduler.ts

@@ -116,6 +116,7 @@ export class ActionScheduler {
      * In most other error cases the manager will try to handle any server errors by itself.
      */
     public async startWithJoin(): Promise<void> {
+        this.state = ActionScheduler.defaultState;
         this._actions = [{ ts: Date.now(), type: MembershipActionType.SendFirstDelayedEvent }];
 
         while (this._actions.length > 0) {
@@ -178,10 +179,6 @@ export class ActionScheduler {
         this.wakeup?.({ replace: [{ ts: Date.now(), type: MembershipActionType.SendScheduledDelayedLeaveEvent }] });
     }
 
-    public resetState(): void {
-        this.state = ActionScheduler.defaultState;
-    }
-
     public resetRateLimitCounter(type: MembershipActionType): void {
         this.state.rateLimitRetries.set(type, 0);
         this.state.networkErrorRetries.set(type, 0);