Skip to content

Remove the Report and ReportBody interfaces #284

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

Merged
merged 1 commit into from
Jun 11, 2025
Merged

Conversation

yoavweiss
Copy link
Contributor

Replaces the obsoleted #258

This change removes the web-exposed interfaces for Report and ReportBody, as there's no use case for them.

@yoavweiss yoavweiss requested a review from domenic June 11, 2025 05:53
@yoavweiss yoavweiss merged commit 1e48c2e into main Jun 11, 2025
3 checks passed
@yoavweiss yoavweiss deleted the remove_interface branch June 11, 2025 06:13
github-actions bot added a commit that referenced this pull request Jun 11, 2025
SHA: 1e48c2e
Reason: push, by yoavweiss

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Jun 11, 2025
SHA: 1e48c2e
Reason: push, by yoavweiss

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
TimvdLippe added a commit to TimvdLippe/webappsec-csp that referenced this pull request Jun 18, 2025
Following the changes in w3c/reporting#284 all report bodies
now need to be dictionaries, following the conclusion of
w3c/reporting#216
mikewest pushed a commit to w3c/webappsec-csp that referenced this pull request Jun 22, 2025
Following the changes in w3c/reporting#284 all report bodies
now need to be dictionaries, following the conclusion of
w3c/reporting#216
dictionary Report {
DOMString type;
DOMString url;
ReportBody? body;

Choose a reason for hiding this comment

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

FYI this does not appear to compile with WebIDL:

  --- stderr
  Traceback (most recent call last):
    File "/home/tvanderlippe/Projects/servo/components/script_bindings/codegen/run.py", line 168, in <module>
      main()
    File "/home/tvanderlippe/Projects/servo/components/script_bindings/codegen/run.py", line 48, in main
      parser_results = parser.finish()
                       ^^^^^^^^^^^^^^^
    File "/home/tvanderlippe/Projects/servo/third_party/WebIDL/WebIDL.py", line 9436, in finish
      production.validate()
    File "/home/tvanderlippe/Projects/servo/third_party/WebIDL/WebIDL.py", line 2402, in validate
      raise WebIDLError(
  WebIDL.WebIDLError: error: Dictionary Report has member with nullable dictionary type, /home/tvanderlippe/Projects/servo/components/script_bindings/codegen/../webidls/ReportingObserver.webidl line 13:14
    ReportBody? body;
                ^
warning: build failed, waiting for other jobs to finish...

Shouldn't type and url be marked as required, and body is implicitly optional?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants