Skip to content

WebMCP: Reject invalid origins with "NotSupportedError" - #61564

Merged
chromium-wpt-export-bot merged 1 commit into
masterfrom
chromium-export-cl-8155899
Jul 28, 2026
Merged

WebMCP: Reject invalid origins with "NotSupportedError"#61564
chromium-wpt-export-bot merged 1 commit into
masterfrom
chromium-export-cl-8155899

Conversation

@chromium-wpt-export-bot

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

See https://crrev.com/c/8121241/comment/a263bec5_46b078a4/ and
webmachinelearning/webmcp#226 (comment).
With both Mason and François in agreement about this, we should
migrate from DataError => NotSupportedError.

R=masonf

Bug: 489045948,536063275
Change-Id: I9e5fc78b082de179c016f8a16dc052744ee28513
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/8155899
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Dominic Farolino <dom@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1669154}

See https://crrev.com/c/8121241/comment/a263bec5_46b078a4/ and
webmachinelearning/webmcp#226 (comment).
With both Mason *and* François in agreement about this, we should
migrate from DataError => NotSupportedError.

R=masonf

Bug: 489045948,536063275
Change-Id: I9e5fc78b082de179c016f8a16dc052744ee28513
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/8155899
Reviewed-by: Mason Freed <masonf@chromium.org>
Commit-Queue: Dominic Farolino <dom@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1669154}

@wpt-pr-bot wpt-pr-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot
chromium-wpt-export-bot merged commit bcbf323 into master Jul 28, 2026
27 checks passed
@chromium-wpt-export-bot
chromium-wpt-export-bot deleted the chromium-export-cl-8155899 branch July 28, 2026 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants