LSP breakpoint placement

[tmat]

Implements #59455
LSP: microsoft/language-server-protocol#1401

[davidwengier]

You can tell who designed the LSP spec when the handler implementation is this simple 😛

[CyrusNajmabadi]

will this break for non-c#/vb @dibarbet shoudl we make this GetService and then return null if it's not available?

[dibarbet]

I don't think its necessary - we should just not advertise the capability for other scenarios

[genlu]

This change has broken ngen in our isnertion FYI @RikkiGibson

02/25/2022 10:03:22.907 [4344]: Failed to load dependency Microsoft.VisualStudio.LanguageServer.Protocol.Internal of assembly Microsoft.CodeAnalysis.EditorFeatures, Version=4.2.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35 because of the following error : The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
02/25/2022 10:03:22.907 [4344]: Failed to load dependency Microsoft.VisualStudio.LanguageServer.Protocol of assembly Microsoft.CodeAnalysis.EditorFeatures, Version=4.2.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35 because of the following error : The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
02/25/2022 10:03:22.923 [4344]: Failed to load dependency Microsoft.VisualStudio.LanguageServer.Protocol.Extensions of assembly Microsoft.CodeAnalysis.EditorFeatures, Version=4.2.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35 because of the following error : The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)

Looks like Microsoft.VisualStudio.LanguageServer.Protocol 17.2.3 isn't inserted in VS or there's issue with ngen time binding redirect.

[dibarbet]

Looks like 17.2.3 insertion is not merged yet - https://devdiv.visualstudio.com/DevDiv/_git/VS/pullrequest/382251