Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Adapt api.Performance.onresourcetimingbufferfull to new events structure #15206

Conversation

queengooborg
Copy link
Collaborator

This PR adapts the resourcetimingbufferfull event of the Performance API to conform to the new events structure.

This PR adapts the resourcetimingbufferfull event of the Performance API to conform to the new events structure.
@queengooborg queengooborg added data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API needs content update 📝 This PR needs a corresponding update to mdn/content to update the documentation labels Mar 1, 2022
@teoli2003 teoli2003 removed the needs content update 📝 This PR needs a corresponding update to mdn/content to update the documentation label Mar 7, 2022
"onresourcetimingbufferfull": {
"__compat": {
"mdn_url": "https://developer.mozilla.org/docs/Web/API/Performance/onresourcetimingbufferfull",
"spec_url": "https://w3c.github.io/resource-timing/#dom-performance-onresourcetimingbufferfull",
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you need to add the spec_url to the other spec_url making it an array. @Elchi3, am I correct?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You are correct, but the spec URL is the exact same for both the event and the event handler, so I have left it unchanged!

@teoli2003 teoli2003 merged commit e977123 into mdn:main Mar 7, 2022
@queengooborg queengooborg deleted the api/Performance/resourcetimingbufferfull/events branch March 7, 2022 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants