sentry-javascript icon indicating copy to clipboard operation
sentry-javascript copied to clipboard

feat(replay): Use unwrapped `setTimeout` to avoid e.g. angular change detection

Open mydea opened this issue 1 year ago • 0 comments

This PR makes sure we use the native, unwrapped setTimeout implementation of the browser. Some environments, e.g. Angular, monkey patch this for their change detection, leading to performance issues (possibly). We have already changed this in rrweb, but we also have some usage of this in replay itself.

This PR should work fine, however all test fail today because we heavily use jest.useFakeTimers(), which basically monkey patches fetch too. So with this change, we do not use the patched timers, leading to everything blowing up 🤯

mydea avatar May 02 '24 07:05 mydea