Add screenshotCacheDuration to FlutterOptions

[denrase]

📜 Description

Cache screenshots in order to reduce CPU impact when calling the SDK in tight (awaited) loops.

I'm up for discussion which, if any, default value should be used. Not its 100ms, which should amount to about 6 frames if the app runs with 60fps.

dart_devtools_2024-11-04_15_42_52.814.json
dart_devtools_2024-11-04_15_44_21.890.json

💡 Motivation and Context

Relates to #2379

💚 How did you test it?

Sample app.

📝 Checklist

• I reviewed submitted code
• I added tests to verify changes
• No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
• I updated the docs if needed
• All tests passing
• No breaking changes

[github-actions]

	Messages
📖	Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against ed17f96

[codecov]

Codecov Report

Attention: Patch coverage is 63.63636% with 4 lines in your changes missing coverage. Please review.

Project coverage is 84.75%. Comparing base (73992eb) to head (ed17f96).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
...rc/event_processor/screenshot_event_processor.dart	63.63%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2387      +/-   ##
==========================================
+ Coverage   84.72%   84.75%   +0.02%     
==========================================
  Files         254       79     -175     
  Lines        9113     2814    -6299     
==========================================
- Hits         7721     2385    -5336     
+ Misses       1392      429     -963     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

Android Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	459.93 ms	524.65 ms	64.71 ms
Size	6.49 MiB	7.57 MiB	1.08 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
e5b744f	302.70 ms	342.17 ms	39.47 ms
0ac1eed	370.60 ms	441.54 ms	70.94 ms
90a08ea	477.25 ms	534.10 ms	56.85 ms
0db91cc	327.85 ms	387.31 ms	59.46 ms
061fed2	434.11 ms	506.49 ms	72.38 ms
4829ad3	381.55 ms	455.45 ms	73.90 ms
519423f	357.00 ms	415.77 ms	58.77 ms
ddc97ad	331.45 ms	384.06 ms	52.61 ms
e3ef570	389.71 ms	459.16 ms	69.45 ms
2d3b03d	309.53 ms	353.40 ms	43.87 ms

App size

Revision	Plain	With Sentry	Diff
e5b744f	6.06 MiB	7.09 MiB	1.03 MiB
0ac1eed	6.06 MiB	7.03 MiB	990.44 KiB
90a08ea	6.49 MiB	7.55 MiB	1.06 MiB
0db91cc	5.94 MiB	6.95 MiB	1.01 MiB
061fed2	6.52 MiB	7.59 MiB	1.06 MiB
4829ad3	6.33 MiB	7.26 MiB	943.11 KiB
519423f	6.06 MiB	7.03 MiB	989.24 KiB
ddc97ad	6.16 MiB	7.14 MiB	1003.75 KiB
e3ef570	6.33 MiB	7.26 MiB	950.38 KiB
2d3b03d	6.06 MiB	7.09 MiB	1.03 MiB

Previous results on branch: feat/cache-screenshots

Startup times

Revision	Plain	With Sentry	Diff
e9bb508	459.20 ms	508.12 ms	48.92 ms

App size

Revision	Plain	With Sentry	Diff
e9bb508	6.49 MiB	7.57 MiB	1.08 MiB

[github-actions]

iOS Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	1238.79 ms	1254.51 ms	15.72 ms
Size	8.38 MiB	9.75 MiB	1.37 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
d4d0807	1246.94 ms	1260.69 ms	13.75 ms
f9d18f3	1240.20 ms	1242.78 ms	2.57 ms
04bd9e6	1230.78 ms	1250.71 ms	19.94 ms
051e97a	1245.94 ms	1249.51 ms	3.57 ms
b8562d0	1249.92 ms	1267.56 ms	17.64 ms
affcf07	1240.61 ms	1266.49 ms	25.88 ms
2e8b1e1	1247.45 ms	1263.67 ms	16.22 ms
0f067d3	1245.71 ms	1269.59 ms	23.88 ms
f1314d5	1246.46 ms	1270.92 ms	24.46 ms
e239c83	1248.40 ms	1269.28 ms	20.89 ms

App size

Revision	Plain	With Sentry	Diff
d4d0807	8.33 MiB	9.64 MiB	1.31 MiB
f9d18f3	8.29 MiB	9.36 MiB	1.07 MiB
04bd9e6	8.33 MiB	9.61 MiB	1.27 MiB
051e97a	8.28 MiB	9.34 MiB	1.06 MiB
b8562d0	8.33 MiB	9.54 MiB	1.22 MiB
affcf07	8.38 MiB	9.70 MiB	1.33 MiB
2e8b1e1	8.33 MiB	9.64 MiB	1.31 MiB
0f067d3	8.32 MiB	9.52 MiB	1.20 MiB
f1314d5	8.10 MiB	9.08 MiB	1004.30 KiB
e239c83	8.38 MiB	9.74 MiB	1.36 MiB

[github-actions]

• 🚫 The changelog entry seems to be part of an already released section ## 8.10.1.
  Consider moving the entry to the ## Unreleased section, please.

[denrase]

Closing in favour of #2368