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.

Set insertSpaceAfterOpeningAndBeforeClosingNonemptyBraces to default to true to keep with existing behavior

Fix https://github.com/Microsoft/monaco-editor/issues/417. Safari 10.1 breaks visibility inheritence.

* Expose insert..Braces rule in VS Code settings

Fixes #18043

* Add requirement blurb for insertSpaceAfterOpeningAndBeforeClosingNonemptyBraces

* Revert tsfmt change

Changed link to use https

This is a very minor change. I just changed the link to stackoverflow to use https instead of http. ^ _ ^

* Nested code lines now no longer display the hover border left.

* Added fix for both light and hc themes.

Fixed gulp task termination before all translations were emitted. Fixed issue with incorrect encoding for pulled translations.

Let the widgetactions debug UI reflect the state of a focused thread