fix: memory leak in pane composite bar

[SimonSiefke]

Fixes a memory leak in paneCompositeBar.

It seems the paneCompositeBar can be visible for a longer period of time, while it's actions can update many times.

private getCompositeActions(){
  // composite actions are updated, but the already registered ones not disposed here
  this.compositeActions.set(compositeId, compositeActions)
}

By changing the code to use a DisposableMap, it ensures that the previous actions are disposed when the actions are updated.

Before

When renaming a variable 97 times with refactor preview, it seems 3 actions are created each time:

After

When renaming a variable 97 times with refactor preview, no more action leaks are detected:

[vs-code-engineering]

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@bpasero

Matched files:

• src/vs/workbench/browser/parts/paneCompositeBar.ts