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

[Chrome] Correct MediaQueryList. #3255

Merged
merged 1 commit into from
Jan 4, 2019
Merged

[Chrome] Correct MediaQueryList. #3255

merged 1 commit into from
Jan 4, 2019

Conversation

jpmedley
Copy link
Contributor

@jpmedley jpmedley commented Jan 4, 2019

The items changed to '45' are from this CL. All other changes are things that should have been copied from the interface level.

A checklist to help your pull request get merged faster:

  • Summarize your changes
  • Data: link to resources that verify support information (such as browser's docs, changelogs, source control, bug trackers, and tests)
  • Data: if you tested something, describe how you tested with details like browser and version
  • Review the results of the linter and fix problems reported (If you need help, please ask in a comment!)
  • Link to related issues or pull requests, if any

@ddbeck ddbeck added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Jan 4, 2019
Copy link
Collaborator

@ddbeck ddbeck left a comment

Choose a reason for hiding this comment

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

Looks good. Thank you! 🎉

@ddbeck ddbeck merged commit 3df4795 into mdn:master Jan 4, 2019
@jpmedley jpmedley deleted the mediaq branch January 4, 2019 18:09
foolip added a commit to foolip/browser-compat-data that referenced this pull request May 11, 2022
Original source: mdn#3255

https://chromium.googlesource.com/chromium/src/+/5fc555b830a9b5b1536e242d8754d7e507bcc75a
was used as the source, which is the right commit but it wasn't first in
Chrome 45 but earlier.

At the time, the MediaQueryList interface had [NoInterfaceObject], but
because MediaQueryListEvent was implemented/exposed at the same time, we
can assume that the versions should match. Chrome 39 was confirmed with
this test:
https://mdn-bcd-collector.appspot.com/tests/api/MediaQueryListEvent

The data for MediaQueryListEvent already matches.

Part of mdn#7844.
queengooborg pushed a commit that referenced this pull request May 11, 2022
Original source: #3255

https://chromium.googlesource.com/chromium/src/+/5fc555b830a9b5b1536e242d8754d7e507bcc75a
was used as the source, which is the right commit but it wasn't first in
Chrome 45 but earlier.

At the time, the MediaQueryList interface had [NoInterfaceObject], but
because MediaQueryListEvent was implemented/exposed at the same time, we
can assume that the versions should match. Chrome 39 was confirmed with
this test:
https://mdn-bcd-collector.appspot.com/tests/api/MediaQueryListEvent

The data for MediaQueryListEvent already matches.

Part of #7844.
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