lateplate
commented
4 years ago - Ups memory require to be 20%. Sentry is showing that the low memory event often occurs prior to the screenshot crashing, and the value is shown as a "level 20". While I can't find any documentation on this, I am bumping the memory requirement a bit to be safe. (We can always bring it back down and monitor, but the priority for now is to minimize crashes however we can).
- Catches/eats any exceptions caused by screenshotting. It doesn't seem like a failure to screenshot should crash the app (for now) until we have more time to investigate. This is a sloppy/global fix, but right now preventing crashes is priority.