Javascript QuickJumpToDefintion throws an error when not called from a javascript file

[fdecampredon]

I have implemented QuickJump in my typescript extension.
However when I try to call quick jump from a TypeScript file, the JavaScript provider is called before mine and throws an error at that line.
The problem is that session is undefined.
By replacing that line this one :

if (!session  || session.editor.getModeForSelection() !== "javascript") {

the error is gone.

[peterflynn]

This basically makes EditorManager.registerJumpToDefProvider() unusable by anyone but the default JS provider...

[peterflynn]

@fdecampredon's suggestion is a good fix -- there are a couple other lesser issues with the same API. I have a local branch with all the fixes rolled up, so I'll put up a PR once S37 is wrapped up.

[njx]

Marking medium priority

[peterflynn]

Turns out I had filed this a while ago -- closing as dupe of #5773. A PR to fix it is now posted.