Right now we have specialized subclasses of this type for different purposes.
However, those subclasses are problematic in terms of OOP as we would need
to have specialized knowledge of all these different types to properly remote them.

Acquire the user's selection up-front if we're on the UI thread for lightbulbs.

Move helper API up as it will be using functionality in EditorFeatures as i refactor more of FAR.

Don't crash in NavigateTo with methods that have __arglist parameters.

Simplify how we go from a text-snapshot to a document in lightbulbs.

Allow properties to be passed with DefinitionItems for FAR.

Track down flakeyness in MoveType tests.

After investigating with jason, we've concluded this strange code was written
back in the days where roslyn workspaces and VS documents were not something
you could trust to be in sync.  So we had to have a lot of painful code to try
to detect and bail out when that happened.  This is definitely not necessary
now, and we can just use the simple and correct function we already hve for this
task.

Remove the synchrounous FindRefs codepaths for C# and VB.

Move the implementation of RQNames down to hte Feature-layer.

There are circumstances where the editor will call into us on hte UI thread and will then
block for results.  Right now this can deadlock as we may have kicked off work to the BG
that then awaits getting the selection back from the UI thread.  To avoid that, if we're on
the UI thread when we start, just go and get the selection and pass it along.

This wasn't previously flagged because this only comes into play if there is an error.  As
written the code will display "$echo isn't a variable" instaed of the actual error.

Fix issue with definite-assignment checking in 'inline-decl' with 'var-typed' locals.

Merge dev15.2.x to master

Removed some unnecessary codegen from async state machine.

Add test for issue https://github.com/dotnet/roslyn/issues/18844

Move helper API up as it will be using functionality in EditorFeatures as part of FAR refactoring work.

Remove unnecessary parsing logic.

The synchronous API still exists for TS to use until they
move over to the entirely async api.

This reverts commit 4f7b313c, reversing
changes made to 6d59ad84.