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

Update Safari support for devicechange_event #8863

Merged
merged 1 commit into from
Jan 26, 2021
Merged

Conversation

Lytigas
Copy link
Contributor

@Lytigas Lytigas commented Jan 26, 2021

Per experiments done by @ddbeck [1], support begins with version 11, aligning with support for ondevicechange.

This seems to contradict what official documentation I can find. Changelogs[2] indicate support was added in technology preview version 72, and the change list for that release indicates support was committed[3] after the tag for Safari 11.1[4], but I'll defer to the experiments.

Fixes #8688

[1] #8688 (comment)
[2] https://webkit.org/blog/8547/release-notes-for-safari-technology-preview-72/
[3] https://trac.webkit.org/changeset/238796/webkit/
[4] https://trac.webkit.org/changeset/230984/webkit

Per work done by @ddbeck [1], support begins with version 11, aligning with support for `ondevicechange`.

[1] mdn#8688 (comment)
@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Jan 26, 2021
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.

Thanks, @Lytigas! I appreciate the extra research you did here. I'm not quite sure what the full story is here, to be honest. The Safari releases are a bit opaque. It's my understanding that it's possible for the WebKit tag to be off or for features to be backported in ways that don't show in changesets (e.g., toggling build flags or flipping default settings). So perhaps the feature existed in 11 and changed in 11.1 and that's what we're seeing here.

For what it's worth, I did an easy check in Sauce Labs (e.g., 'ondevicechange' in navigator.mediaDevices returns true in Safari 11.0) to make sure this still seems plausible. Since it looks OK—and it's definitely more helpful than showing false—I'm going to go ahead with approving and merging this for now, with the qualification that I'm not closing the door to a follow up PR that amends this (hopefully with some comprehensive test cases that help us understand what the full story is).

@ddbeck ddbeck merged commit 3938875 into mdn:master Jan 26, 2021
@Lytigas Lytigas deleted the patch-1 branch January 27, 2021 03:54
germain-gg pushed a commit to germain-gg/browser-compat-data that referenced this pull request Feb 1, 2021
…icture

* upstream/master: (1123 commits)
  Remove Chromium 89 from String.at / Array.at / TypedArray.at (mdn#8869)
  Add worker_support info for CacheStorage (mdn#8783)
  Remove several needless "Enabled by default" notes (mdn#8899)
  Add HTML global attribute nonce (mdn#8764)
  api.Navigator.vibrate - Firefox for Android doesn't vibrate (mdn#7172)
  Mark MediaSource's onsourceclose as not supported in Firefox (mdn#8881)
  Update Florian's ownership (mdn#8893)
  Mention fix for Chrome's broken PDF loading (mdn#8867)
  Fill out Chrome data for html.elements.source.{sizes,srcset} (mdn#8889)
  Weekly data release for 2021-01-28
  Add text-decoration-thickness for Opera 73+ (mdn#8872)
  Update :is and :where pseudo-classes for Chrome (mdn#7375)
  Add note re Safari <9 partial srcset/sizes support (mdn#7353)
  Update data for when href (not xlink:href) can be used in SVG (mdn#6603)
  Add top-level await (mdn#8807)
  TouchList: Add Safari Desktop and Safari iOS versions (mdn#8848)
  Update Firefox versions to account for Firefox 85 release (mdn#8864)
  Fix page_action.show_matches support for Android (mdn#8844)
  Update Safari support for devicechange_event (mdn#8863)
  Add HTTPS-only to privacy.network (mdn#8830)
  ...
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.

api.MediaDevices.devicechange_event - Safari Support
2 participants