memory leak in scheduleTask

[BryanCrotaz]

weak map is not being cleaned up.

getTimersDisposable leaves an entry in the registeredTimers WeakMap and individual cancelIds are left after tasks have completed

[BryanCrotaz]

Doesn't leak:

  _openHandler() {
    if (this.onOpened) {
      run(() => {
        schedule('sync', () => {
          if (!this.isDestroyed && !this.isDestroying && this.onOpened) {
            this.onOpened(this, this.url);
          }
        });
      });
    }
  }

Leaks:

   _openHandler() {
    if (this.onOpened) {
        scheduleTask(this, 'sync', () => {
          if (this.onOpened) {
            this.onOpened(this, this.url);
          }
        });
    }
  }
...
  destroy() {
    runDisposables(this);
 
    this.close();
    this.onOpened = null;
    this.onClosed = null;
    this.onError = null;
    this.onMessage = null;

    this._super();
  }

[BryanCrotaz]

I think the cancelId is being left in the registeredTimers array once the scheduled execution is complete. This is a problem for long lived objects.