Confusing definition found for functions in some cases

[elibarzilay]

goToDefinition handles functions specially, leading to this confusing highlight:

let f1, f2;
f2 = f1 = function () {};
//   ^^
f1();

this is different from:

let g1, g2;
g2 = g1 = () => {};
//        ^^^^^^^^
g1();

which seems like a better choice in the former case too.

This is an issue that was raised in #38567 (which removes the first f1/g1 from the results but doesn't change the value highlights).