Fix duplicate/inaccurate results in LSP GoToImpl

Use MEF instead of direct construction

Add feature to remove confusing use of suppression operator

* Add reqired API for moving SBD analyzer OOP

* Add the missing lisense header

* Add another missing lisense header

* Simplify some wrapper types

* Restore all the delete & edit API, mark them as Obsolete. Add a callback interface

* Fix typo

* Obselete some more APIs

* Add a dispose API

* Obselete more wrapper

* Remove the unused wrapper

* Remove a unused method

Embed generated source texts in the portable PDB:
- Allows stepping in etc. in the IDE as well as a dump debugging after compilation
- Consistent ordering of generated syntax trees.
- Unique hint names
- Restrict hint names to file path legal characters
- Add tests

Update src/Features/CSharp/Portable/RemoveUnnecessarySuppression/CSharpRemoveUnnecessarySuppressionCodeFixProvider.cs
Co-authored-by: NManish Vasani <mavasani@microsoft.com>

Fix fading locations for unnecessary code.

Potentially fixed a bug regarding static items not bolding

Ensure that TodoCommentOptions are serialized to OOP

Fixes #43788
TodoCommentOptions and TodoCommentOptionsProvider were defined in EditorFeatures layer and we only serialize the options defined in Workspaces and Features layer to the OOP side: https://github.com/dotnet/roslyn/blob/287a25a51324f252e61b6f2186e7df60c8a6161b/src/Workspaces/Core/Portable/Options/GlobalOptionService.cs#L79-L85

In future, we either need to add support to serialize all changed options to OOP (could be expensive) or somehow enforce that all the OOP related options are defined in Workspaces and Features layer.

Remove build enforcement of unused usings in IDE layers

Remove redundant helper.

We had enabled this rule as a build time enforcement for IDE projects, but it has caused bunch of build breaks and it turning out to be too onerous to enforce. It was decided that this would be better implemented as a post-build code cleanup bot.

Watch directories for source files and some metadata references

Co-authored-by: NJason Malinowski <jason@jason-m.com>

Tweak .editorconfig legacy engine controls

Ensure that we show all applicable refactorings for lightbulb under t…

* Rename

* Change the name to Pascal style

* Rename the enum and change the wrong string

We originally offered this checkbox as a way for users who were broken
by our .editorconfig support to have an opt-out while we diagnosed what
issues they were running into. At this point we aren't tracking any
bugs, so we want anybody who had checked this checkbox to get the new
experience again in hopes that their problems are now addressed. If we
don't hear any additional feedback, we'll remove the legacy engine
entirely in 16.8.

This constant was no longer used.

Refactor some of Quick Info to prep for LSIF Quick Info support