performance.timerify(fn) behave inconsistently for sync/async functions

### Version master branch ### Platform All ### Subsystem perf_hooks ### What steps will reproduce the bug? ```js function f1() { throw new Error() } async function f2() { throw new Error() } let h1 = perf_hooks.createHistogram() let h2 = perf_hooks.createHistogram() let g1 = perf_hooks.performance.timerify(f1, {histogram: h1}) let g2 = perf_hooks.performance.timerify(f2, {histogram: h2}) g1() await g2() h1.count === h2.count // expect true, actual false ``` ### How often does it reproduce? Is there a required condition? Always ### What is the expected behavior? Timerified functions should behave consistently for whether trigger `histogram.record`. ### What do you see instead? Currently, normal function would not trigger `histogram.record` if throw; but async function (or any function which return promise) would. ### Additional information Currently, the code is like `if (typeof result?.finally === 'function') return result.finally(...)`. It should use `result.then` instead, so the behavior will consistent with non-thenable result. Even we want to always trigger `histogram.record`, we should not use `finally` because thenable is only require to have `then` method, not `finally` method.