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 PresentationConnection API to new events structure #15185

Merged
merged 1 commit into from
Mar 4, 2022

Conversation

queengooborg
Copy link
Contributor

This PR adapts the PresentationConnection API to conform to the new events structure.

Note: there are no MDN pages associated with this event, so there will be no corresponding content PR. Any broken MDN URLs in BCD have been removed.

This PR adapts the PresentationConnection API to conform to the new events structure.

Note: there are no MDN pages associated with this event, so there will be no corresponding content PR. Any broken MDN URLs in BCD have been removed.
@queengooborg queengooborg added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Feb 28, 2022
"support": {
"chrome": {
"version_added": "47"
"version_added": "50"
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't match the parent interface and made me suspect that on* properties were missing while the events were actually supported, but per https://source.chromium.org/chromium/chromium/src/+/5458a1b5f82ef7a0d01fca7c7352cd7fe7ec5d9a the events and the properties were added together. PresentationConnectionCloseEvent is also marked as Chrome 50.

@foolip foolip merged commit 134c914 into mdn:main Mar 4, 2022
@queengooborg queengooborg deleted the api/PresentationConnection/events branch March 4, 2022 14:13
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