Containers not being removed after debugging when using hotkeys to restart debugging

[Technoguyfication]

Summary:

When using removeContainerAfterDebug in my launch.json, containers aren't being removed after debugging when using the Ctrl+Shift+F5 hotkey to restart debugging. They're removed just fine when clicking the restart button, just not when using the hotkey.

How to reproduce:

1. Set up docker container debugging with vscode (I'm using node.js)
2. Set removeContainerAfterDebug to true in launch.json in the launch configuration object
3. Start a container and begin debugging
4. Restart debugging using the hotkey (Ctrl+Shift+F5)
5. The previous container is not removed as it should be

Expected behavior:

The container should be removed after debugging, the same as when you click the restart debugging button with the mouse.

[bwateratmsft]

Could be related to how VSCode handles tasks; we'll have to investigate.

[bwateratmsft]

I've investigated and found that this issue is external: microsoft/vscode#117984

When restarting debugging via the UI button, vscode.debug.onDidTerminateDebugSession does get called. However, when restarting with Ctrl+Shift+F5, it does not get called, and we do not shut down the container.

[bwateratmsft]

Reopening. The root problem is actually in this extension; we misunderstood how the restart flow works.

[bwateratmsft]

This fix is now released with Docker extension version 1.12.0.