set supportsLoadedSourcesRequest capability if DA implements the request

[weinand]

The new built-in loaded scripts explorer (which will appear in August) honors the supportsLoadedSourcesRequest capability and only shows the "Loaded Scripts" view if the value is true.

Please add this to the initializeRequestmethod:

	// This debug adapter supports loaded sources request
	response.body.supportsLoadedSourcesRequest = true;

This applies to the debug types: node2, extensionHost, and chrome.

I suggest to include this fix already in July because I will enable the loaded scripts explorer in Insiders as soon as the July release is out of the door.

For details please see microsoft/vscode#37767

[weinand]

@roblourens I see this change in 1.26.8 but no longer in master.
Was there a problem with it?

[roblourens]

It's also in master, where are you looking?

[weinand]

The capability is missing in the current Insiders and this commit removes it: a131f45
probably because it it now provided by vscode-chrome-debug-core.

It is not important for 1.26, but I was just wondering why the now built-in loadedscripts view no longer shows up...

[roblourens]

This commit from a few days ago put it back: https://github.com/Microsoft/vscode-node-debug2/blob/master/src/nodeDebugAdapter.ts#L78

[weinand]

Sorry, but I do not see the supportsLoadedSourcesRequest in https://github.com/Microsoft/vscode-node-debug2/blob/master/src/nodeDebugAdapter.ts#L78:

[roblourens]

Sorry I was reading something completely different

[roblourens]

Yes it is provided by -core now

[weinand]

...and the 1.26 version still uses an older core which doesn't have the capability.

[roblourens]

I'll publish that for master now

[roblourens]

microsoft/vscode@271a56a