Skip to content

Edge ≤79 -> mirror#27177

Merged
caugner merged 5 commits intomainfrom
edge-79-range
Jul 17, 2025
Merged

Edge ≤79 -> mirror#27177
caugner merged 5 commits intomainfrom
edge-79-range

Conversation

@queengooborg
Copy link
Copy Markdown
Contributor

@queengooborg queengooborg commented Jun 30, 2025

This PR changes all of the statements for Edge ≤79 to mirror from Chrome. Based upon commit history and the data, it seems that most likely this data was all mirrored from Chrome via the original mirroring script anyways.

@github-actions github-actions bot added infra Infrastructure issues (npm, GitHub Actions, releases) of this project data:http Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML linter Issues or pull requests regarding the tests / linter of the JSON files. data:svg Compat data for SVG features. https://developer.mozilla.org/docs/Web/SVG size:l [PR only] 101-1000 LoC changed labels Jun 30, 2025
@github-actions github-actions bot removed infra Infrastructure issues (npm, GitHub Actions, releases) of this project linter Issues or pull requests regarding the tests / linter of the JSON files. labels Jun 30, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jun 30, 2025 •
edited
Loading

Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs).

caugner
caugner reviewed Jul 1, 2025
View reviewed changes
caugner
caugner reviewed Jul 1, 2025
View reviewed changes
@github-actions github-actions bot added the merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. label Jul 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 8, 2025

This pull request has merge conflicts that must be resolved before it can be merged.

@caugner caugner mentioned this pull request Jul 8, 2025
Merged
@github-actions github-actions bot removed the merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. label Jul 8, 2025
@caugner
Copy link
Copy Markdown
Contributor

caugner commented Jul 8, 2025 •
edited
Loading

Looking at the Diff (by change, mirroring applied), there are two changes remaining that make me hesitate:

  1. Can we set svg.elements.{foreignObject,set} to Edge 12+ in a separate PR?
  2. (✅) Do we know why api.RTCPeerConnection.RTCPeerConnection.configuration_iceServers_parameter.credentialType_parameter (now set to false) was ever marked as supported?

Edit: Looked into 2 -> This was added in 963066c#diff-a1175fb96f7f18f409882d748645c12ccd49df8d12700451a24e8248d5f609e3R109, when Chrome support was saying true, which was later changed to false in 561158b#diff-a1175fb96f7f18f409882d748645c12ccd49df8d12700451a24e8248d5f609e3R106.

@caugner
Copy link
Copy Markdown
Contributor

caugner commented Jul 11, 2025

@queengooborg See my previous comment in case you missed it.

@github-actions github-actions bot added the merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. label Jul 15, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 15, 2025

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot removed the merge conflicts 🚧 This PR needs to merge latest "main" branch to resolve a merge conflict or other issue. label Jul 15, 2025
caugner
caugner approved these changes Jul 17, 2025
View reviewed changes
Copy link
Copy Markdown
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.

Thanks, this looks good to me now!

@caugner caugner merged commit 64fa66f into main Jul 17, 2025
11 checks passed
@caugner caugner deleted the edge-79-range branch July 17, 2025 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@caugner caugner caugner approved these changes

Assignees

No one assigned

Labels

data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:css Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML data:http Compat data for HTTP features. https://developer.mozilla.org/docs/Web/HTTP data:svg Compat data for SVG features. https://developer.mozilla.org/docs/Web/SVG size:l [PR only] 101-1000 LoC changed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@queengooborg @caugner