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 Chromium versions for CSPViolationReportBody API #18540

Closed

Conversation

queengooborg
Copy link
Contributor

This PR updates and corrects version values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the CSPViolationReportBody API, based upon results from the mdn-bcd-collector project (v7.1.2).

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/api/CSPViolationReportBody

Check out the collector's guide on how to review this PR.

Note: this was originally set to "74" in #10925 based upon commit history. However, while Chrome seems to have IDL for this feature, it's not exposed.

This PR updates and corrects version values for Chromium (Chrome, Opera, Samsung Internet, WebView Android) for the `CSPViolationReportBody` API, based upon results from the [mdn-bcd-collector](https://mdn-bcd-collector.gooborg.com) project (v7.1.2).

Tests Used: https://mdn-bcd-collector.gooborg.com/tests/api/CSPViolationReportBody

_Check out the [collector's guide on how to review this PR](https://github.com/GooborgStudios/mdn-bcd-collector#reviewing-bcd-changes)._

Note: this was originally set to "74" in mdn#10925 based upon commit history.  However, while Chrome seems to have IDL for this feature, it's not exposed.
@queengooborg queengooborg added data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API not ready This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. info needed This needs more information to review or act on. labels Jan 5, 2023
@queengooborg
Copy link
Contributor Author

After researching this further, I found that Chrome actually does support this, but it's not directly exposed like it should be. I created mdn/content#23433 to add an example on how to obtain an instance of this interface, and I pushed openwebdocs/mdn-bcd-collector@c0161af to disable this interface in the collector for now (until I can figure out how to implement a test in the collector).

@queengooborg queengooborg deleted the api/CSPViolationReportBody/chrome branch January 6, 2023 11:59
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 info needed This needs more information to review or act on. not ready This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant