[Master] fix 16407 - LS in binding element of object binding pattern

[yuit]

Fix #16407
Because identifier in bindingElement in objectBindingPattern is considered declaration name, previously it will just return its symbol. So when perform goto-definition, it will simply just stay where it is

function bar<T>(onfulfilled: (value: T) => void) {
  return undefined;
}

interface Test {
  prop2: number
}
bar<Test>(({pr/*gotoDef*/op2})=>{});  // just stay at the same place

However, it is desirable to actually goto the prop2 in an interface. See: master...master-fix16407#diff-3c46922136bdd65e7a2c2d12db87bbe5....

• Updates change in .symbols files and .types file as we will refer the symbol to the object type now

[weswigham]

I've iterated on yui's change - it no longer changes how we match up symbols in the checker, as that was breaking flow control analysis. Now, it does for object binding patterns a similar action as it does for property names in an object literal. The baseline changes are now nonexistent, and it is just new services functionality.

[amcasey]

Would you mind rephrasing this a little? I find it a bit hard to follow.

[amcasey]

Alternatively, node === (node.parent.propertyName || node.parent.name).

[weswigham]

@amcasey done

[amcasey]

LGTM