Queuing a large number of WinJS promises that aren't actually async can lead to a stack overflow. Also this is faster.

Fixes #24688

Fixes #24630
Related #19665

Remove OutputService reference from SearchService, this should be handled elsewhere

Tested recomppiling VSCode with TS2.3. This fixes a few issues that were discovered, including workarounds for:

* https://github.com/Microsoft/TypeScript/issues/15184
* https://github.com/Microsoft/TypeScript/issues/15185

For debugging purposes and transparency.

Fixes #24209

**Bug**
Currently, for the js/ts references code lens, even if there are zero references you can click on the lens. This display an empty peek view

**Fix**
Allow code lenses to only register a title for the lens with no actual backing command

fixes #24534