Preserve whitespace in debug hover expresion value

Part of #74250

- Extend `Disposable` in classes where it makes sense
- Use `DisposableStore` for lists of disposables
- Make `combinedDisposable` take arguments instead of an array (so that you can't pass in an array and then modify the array after the fact)

Not sure why incremental builds missed this

Fixes #74242

Our current usage of dispoable arrays can leak disposables (see #74242 for details). This change introduces a new class called `DisposableStore` that can be used mostly as a drop in replacement for an array of disposables but will not leak disposbles if it is disposed

`DisposableStore` was extracted from the existing `Dispoable` class, which already implements this pattern. However `Disposable` is intended to be used as a base class while `DispoableStore` is a value class.

In addition, this change hides the `toDispose` / `_toDipose` members of `Disposable` as direct write access to the disposable array also allows for leaks (and breaks encapsulation)

Fixes #74191

Instead of using a base64 encoded data uri to display images, use the fileService to read Buffer, convert this to a Blob, and display the blob

For #72824

Add rnpm-plugin-windows and pom.xml check

Fixes #74164

fix #74085

Comment API

Input box escape \n, \r etc in the setter

Fixes microsoft/vscode-remote-release#42
Fixes microsoft/vscode-remote-release#170

* Add universal list collapseAll keybinding

This adds the ability to perform a collapse all on a full tree upon a keyboard shortcut. Upon performing the shortcut, it will collapse every node on the tree.

debt - fs.readlink() can return relative paths, so we use fs.realpath() instead to ensure absolute paths for symbolic links when watching.

update master

microsoft/vscode-remote-release/issues/2 - ignore machine settings in user settings in remote context