Michael Dellanoce

this _might_ have an impact on performance when things do make it to the addList though, since the node order is reversed, and addList is traversed from the tail node...

Just had this happen on another customer's site, so seems somewhat common for Angular apps.

zone.js strikes again... added a similar commit for addEventListener. This one wasn't a performance issue, but the passive mousemove listener added by rrweb forces _all_ mousemove listeners added through zone.js...

@Juice10 I updated rrweb-snapshot's setTimeout calls. Those seem unlikely to lead to endless looping like what I've seen with hookSetter, but I guess it doesn't hurt. > This PR doesn't...

a test failed, but i'm not sure why...

added a similar fix for requestAnimationFrame in ProcessedNodeManager. If #1328 ever merges that commit can be dropped

@Mark-Fenng @Juice10 thanks for the feedback, totally understand the performance concerns. I think the maskAllText suggestion will work, and I have a few other ideas. I'll see what I can...

@YunFeng0817 @Juice10 sorry for the delay, but I added 3 commits to address performance: 1. if maskAllText is true, and unmasking either is not configured (best case) or fails to...

@Juice10 I added 2 new benchmarks for masking and unmasking. Here are the results from master:     Note: the unmasking test on master is effectively the same...

@Juice10 ready for another look, i think