🐛 [RUM-94] ignore performance resource timings with negative duration

[BenoitZugmeyer]

Motivation

Starting with Chrome 128, blob URLs are reported as PerformanceResourceTiming entries. Those entries have (always?) negative durations, which is unexpected. The intake rejects RUM Resource events with negative duration already, so sending them is useless and breaks our e2e tests when validating the events against the rum-events-format.

Changes

Let's ignore resource entries with negative duration.

Note: I opened a Chromium issue about this, but in the meantime, there is absolutely no value in sending those entries to the intake.

This PR also rename a bunch of "PerformanceResourceTiming"-related function to have consistent naming.

Testing

• Local
• Staging
• Unit
• End to end

---

I have gone over the contributing documentation.

[BenoitZugmeyer]

/to-staging

[dd-devflow]

🚂 Branch Integration: starting soon, median merge time is 0s

Commit a32799d88a will soon be integrated into staging-35.

Use /to-staging -c to cancel this operation!

[dd-devflow]

🚂 Branch Integration: This commit was successfully integrated

Commit a32799d88a has been merged into staging-35 in merge commit a4509b1255.

Check out the triggered pipeline on Gitlab 🦊

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 96.42857% with 1 line in your changes missing coverage. Please review.

Project coverage is 93.43%. Comparing base (e74e435) to head (3f1a692).
Report is 16 commits behind head on main.

Files with missing lines	Patch %	Lines
...ages/rum-core/src/domain/resource/resourceUtils.ts	92.85%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2958   +/-   ##
=======================================
  Coverage   93.42%   93.43%           
=======================================
  Files         271      271           
  Lines        7637     7641    +4     
  Branches     1704     1706    +2     
=======================================
+ Hits         7135     7139    +4     
  Misses        502      502           

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

[cit-pr-commenter]

Bundles Sizes Evolution

📦 Bundle Name	Base Size	Local Size	𝚫	𝚫%	Status
Rum	160.86 KiB	160.95 KiB	88 B	+0.05%	✅
Logs	55.98 KiB	55.98 KiB	0 B	0.00%	✅
Rum Slim	109.47 KiB	109.56 KiB	88 B	+0.08%	✅
Worker	25.21 KiB	25.21 KiB	0 B	0.00%	✅

🚀 CPU Performance

Action Name	Base Average Cpu Time (ms)	Local Average Cpu Time (ms)	𝚫
addglobalcontext	0.001	0.001	0.000
addaction	0.032	0.035	0.003
addtiming	0.001	0.001	0.000
adderror	0.033	0.033	0.001
startstopsessionreplayrecording	0.973	1.033	0.060
startview	0.931	0.919	-0.012
logmessage	0.020	0.021	0.001

🧠 Memory Performance

Action Name	Base Consumption Memory (bytes)	Local Consumption Memory (bytes)	𝚫 (bytes)
addglobalcontext	8.35 KiB	8.25 KiB	-103 B
addaction	39.97 KiB	40.94 KiB	995 B
addtiming	7.50 KiB	7.75 KiB	257 B
adderror	46.69 KiB	45.08 KiB	-1647 B
startstopsessionreplayrecording	5.11 KiB	6.55 KiB	1.44 KiB
startview	316.61 KiB	321.83 KiB	5.22 KiB
logmessage	41.45 KiB	41.15 KiB	-314 B

🔗 RealWorld