🐛 [RUM-1666] Don't set negative action loading time

[RomanGaignault]

Motivation

We noticed that we have actions with negative actions negative loading time. This is probably due to the fact that Date.now() is not monotonic and sometimes goes “back in time”

Changes

If that's the case, we could just not set the loading time of the action event.

Testing

• Local
• Staging
• Unit
• End to end

---

I have gone over the contributing documentation.

[cit-pr-commenter]

Bundles Sizes Evolution

📦 Bundle Name	Base Size	Local Size	𝚫	𝚫%	Status
Rum	157.94 KiB	157.94 KiB	4 B	0.00%	✅
Logs	56.42 KiB	56.42 KiB	0 B	0.00%	✅
Rum Slim	104.43 KiB	104.43 KiB	4 B	0.00%	✅
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)	𝚫
adderror	0.033	N/A	N/A
addaction	0.016	N/A	N/A
logmessage	0.005	0.005	0.001
startview	0.961	0.946	-0.016
startstopsessionreplayrecording	0.772	0.743	-0.028
addtiming	0.001	0.001	0.000
addglobalcontext	0.002	0.001	-0.000

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.31%. Comparing base (f221bc9) to head (d053b71).
Report is 5 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2764   +/-   ##
=======================================
  Coverage   93.31%   93.31%           
=======================================
  Files         241      242    +1     
  Lines        7062     7062           
  Branches     1561     1561           
=======================================
  Hits         6590     6590           
  Misses        472      472           

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

[RomanGaignault]

/to-staging

[dd-devflow]

🚂 Branch Integration: starting soon, merge in < 0s

Commit 0eb757a7d1 will soon be integrated into staging-21.

This build is going to start soon! (estimated merge in less than 0s)

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

[dd-devflow]

🚂 Branch Integration: This commit was successfully integrated

Commit 0eb757a7d1 has been merged into staging-21 in merge commit 2668533c61.

Check out the triggered pipeline on Gitlab 🦊