I've seen a few places where we are wrapping things that could be sync
or async, and we want to do something with the return value. By adding
an onSuccess handler to `handelCallbackErrors` we can handle this more
generically in the future:
```js
const wrapped = handleCallbackErrors(
 () => fn(), 
 // error handler
 (_err) => {}, 
 // finally handler
 () => {},
 // success handler, gets value or resolved value if function returns a promise
 (_value) => {}
);
```
While scanning a bit for places where we could already use this, I also
found two bugs around this, opened separate PRs for them:
* #17680
* #17681