* add callstack on debug to make investigation easier

* if GetPinnedScopeAsync throws cancellation exception, connection can be leaked not disposed.

fixed the issue and added test

* make sure cancellation is thrown in test

* PR feedbacks

* more feedbacks

* address PR feedback

* enable logAnalyzerExecutionTime on IDE so that we can start track analyzer perf

* removed unnecessary ICodeAnalysisDiagnosticAnalyzerExecutor interface.

it was added when OOP is first introduced to make sure VS.Next dll won't get loaded to VS process if OOP is not enabled.

when it is enabled by default, rather than removing the interface, implementation just moved down to feature layer to reduce code churn.

now, I am properly removing unnecessary abstraction.

* take Executor out of test MEF composition

* added IRemoteDiagnosticAnalyzerService interface

* made initial version working.

* added tests

* add tracking for inproc only analyzers

* pass in diagnostic analyzer service

* added open file performance tracking as well.

* added PII test

* dont hash analyzerId when it is reported by internal user

* added link to LOF wiki

* made blame to track open files as well.

* forgot to add return in if statement

* reduce threshold to 100ms

decide to start from lower threshold and then iterate rather than start from higher threshold

* added a way to log real time perf data in local machine with explicit option which can be used to train formula later

* addressed ivan's feedbacks

* renamed to ExpensiveAnalyzerInfo

* addressed PR feedbacks

* more renames

* addressed PR feedbacks.

renamed as much as I can find.

* listener can be null in unit test

* addressed PR feedbacks

fixed issue where same analyzer reference is installed in both vsix and nuget. deduplication in inproc analyzer manager wasn't present in OOP case. (#23772)

this doesn't address our workitem to merge those two. that work is a separate work item. this just follow what we currently do in proc.

renamed GetRemoteHostClientAsync to TryRemoteHostClientAsync to make it clear that it can return null when remote host is not available.

left GetRemotehostClientAsync there as Obsolete so that this doesnt break other team such as LUT.

will remove it once other team moved to new bits.

now, it will get synched when remote host is first created rather than doing that on demand. so functionality change, so re-organized where things happening.

move down diagnostic analyzer executor down to workspace layer since all requires one now lives in workspace layer.