Skip to content

Update Node.js data for PerformanceEntry/Mark/Measure #25008

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

skyclouds2001
Copy link
Contributor

@skyclouds2001 skyclouds2001 commented Nov 8, 2024

Summary

most performance related features has been exposed to global via nodejs/node#44483, in node v19.0.0

docs indicate that PerformanceMark and PerformanceMeasure is implemented in v18.2.0, v16.17.0, not v8.5.0
docs changes happens in nodejs/node#44483
PerformanceMark is implemented and can be accessed via perf_hooks module in nodejs/node#37136, in v16.0.0
PerformanceMeasure is implemented in nodejs/node#37136, in v16.0.0; bu can be accessed via perf_hooks module in nodejs/node#39297, in v16.7.0

manually test in local node instance:

image

image

so fired nodejs/node#55792 at node's repo

Test results and supporting details

https://nodejs.org/docs/latest/api/globals.html#performanceentry
https://nodejs.org/docs/latest/api/globals.html#performancemark
https://nodejs.org/docs/latest/api/globals.html#performancemeasure
https://nodejs.org/docs/latest/api/perf_hooks.html#class-performanceentry
https://nodejs.org/docs/latest/api/perf_hooks.html#class-performancemark
https://nodejs.org/docs/latest/api/perf_hooks.html#class-performancemeasure

Related issues

@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Nov 8, 2024
@skyclouds2001 skyclouds2001 marked this pull request as ready for review November 11, 2024 11:33
@caugner caugner added the nodejs Data about Node.js label Nov 12, 2024
@github-actions github-actions bot added the size:m [PR only] 25-100 LoC changed label Nov 23, 2024
Co-authored-by: Queen Vinyl Da.i'gyu-Kazotetsu <vinyldarkscratch@gmail.com>
Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you, just a typo.

@caugner caugner dismissed queengooborg’s stale review January 6, 2025 18:11

Changes applied.

@caugner caugner changed the title Update nodejs data for some performance features Update Node.js data for PerformanceEntry/Mark/Measure Jan 6, 2025
@caugner caugner merged commit ddc90cf into mdn:main Jan 6, 2025
9 checks passed
@skyclouds2001 skyclouds2001 deleted the node-performanceentry-performancemark-performancemeasure branch January 7, 2025 14:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API nodejs Data about Node.js size:m [PR only] 25-100 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants