Execute revealFileInOS on TreeItem will use current focused file's URI

[Vigilans]

Issue Type: Bug

This is a regression bug after 0dbe568 which fixes #86103:

export function getMultiSelectedResources(resource: URI | object | undefined, listService: IListService, editorService: IEditorService, explorerService: IExplorerService): Array<URI> {
    const list = listService.lastFocusedList;
    if (list?.getHTMLElement() === document.activeElement) {
        // Explorer
        if (list instanceof AsyncDataTree) {  // <-- TreeView List will also fall into this block!
            // Explorer
            const context = explorerService.getContext(true);
            if (context.length) {
                return context.map(c => c.resource);
            }
        }

When executing getMultiSelectedResources with listService derived from TreeView, if will fall into the logic which only handles FileExplorer, causing the returned resource to be the currently focused file's uri.

Steps to Reproduce:

1. Consider a java-dependency-viewer treeview:
   

2. Execute Reveal In Explorer on a jar file, which will forward jar file's uri to revealFileInOS command.

3. On older versions, it will work as expected, but in insider version and the newly released version, it will open the folder containing the currently focused file.

VS Code version: Code 1.41.0 (9579eda, 2019-12-11T18:32:17.711Z)
OS version: Linux x64 4.15.0

[isidorn]

@Vigilans vscode insiders from next week will contain your fix. Just let us know if the issue gets resolved for you so we can verify this issue. Thank you

[Vigilans]

Hi @isidorn, I confirmed that this issue has been resolved in new insider version, thank you!

[isidorn]

Thank you, adding verified label.