Report updateGraph time in async requests (that is getErr) as well as part of request completed event

[sheetalkamat]

I think this was over site when we added this to Respose and forgot about "getErr" being async request

[typescript-bot]

Thanks for the PR! It looks like you've changed the TSServer protocol in some way. Please ensure that any changes here don't break consumers of the current TSServer API. For some extra review, we'll ping @sheetalkamat, @mjbvz, @zkat, and @joj for you. Feel free to loop in other consumers/maintainers if necessary.

[typescript-bot]

Looks like you're introducing a change to the public API surface area. If this includes breaking changes, please document them on our wiki's API Breaking Changes page.

Also, please make sure @DanielRosenwasser and @RyanCavanaugh are aware of the changes, just as a heads up.

[sheetalkamat]

@mjbvz I am not sure if we need this or not because as I looked into vscode code, currently this is logged into telemetry for all requests but just for completions.
Not sure if this is interesting and useful or not. This came up when @gabritto and i were discussing reporting duration of diagnostic event.