Dominic Farolino (Gerrit)

unread,

Jun 16, 2026, 12:21:27 PM (12 days ago) Jun 16

to Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, Fr, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Fr

Dominic Farolino voted and added 5 comments

Votes added by Dominic Farolino

Commit-Queue

+1

5 comments

File chrome/browser/flag_descriptions.h

Line 3176, Patchset 5 (Parent):inline constexpr char kWebMCPTestingName[] = "WebMCP for testing";

Fr . resolved

Can we keep the name "WebMCP for testing"? See https://www.google.com/search?q=%22WebMCP+for+testing%22 for instance

Dominic Farolino

Hmm, I'd prefer not to, but it can't hurt to keep the name the same, so that sounds fine.

File third_party/blink/renderer/core/script_tools/model_context.h

Line 179, Patchset 1: ExecutionContext* GetExecutionContext() const override;

Fr . resolved

Can we remove this? I believe it was used only for `ModelContextTesting::GetExecutionContext()`

Dominic Farolino

No, `EventTarget` lists this method as a pure virtual function.

Line 11, Patchset 1:#include "base/functional/callback_forward.h"

Fr . resolved

Do we still need this?

Dominic Farolino

Negative. Nice catch!

File third_party/blink/renderer/core/script_tools/model_context_testing.idl

Line 16, Patchset 1 (Parent): RuntimeEnabled=WebMCPTesting

Fr . resolved

The blink runtime feature `WebMCPTesting` needs to be removed as well. Note that you'll need to update https://source.chromium.org/chromium/chromium/src/+/main:chrome/browser/about_flags.cc;l=12539;drc=b49ce6f9adf6e7151b06168880212578150f57e4 as well to switch to `blink::features::kWebMCP`. The flag name needs to stay the same.

Dominic Farolino

Why does the name need to stay the same? To stay consistent with published developer documentation or something? Anyways, I've done this so I think we're clear!

File third_party/blink/renderer/platform/runtime_enabled_features.json5

Line 6866, Patchset 5 (Parent): name: "WebMCPTesting",

Fr . resolved

If we're removing this feature, it means we'll have to update [Puppeteer](https://pptr.dev/guides/webmcp), [Chrome DevTools for Agent](https://github.com/ChromeDevTools/chrome-devtools-mcp), and maybe more documentation. Is there a way somehow to keep this functional?

Dominic Farolino

I can hold off on removing it for now, but we should remove it soon. Can you kick off the process of changing whatever you'd want to change for us to be able to remove this? I'll try and remember to remove it in the next milestone.

Open in Gerrit

Related details

Attention is currently required from:

Fr

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Dominic Farolino (Gerrit)

unread,

Jun 16, 2026, 2:28:20 PM (12 days ago) Jun 16

to kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, Fr, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Fr

Dominic Farolino voted Commit-Queue+1

Commit-Queue

+1

Open in Gerrit

Related details

Attention is currently required from:

Fr

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Fr (Gerrit)

unread,

Jun 17, 2026, 3:56:48 AM (12 days ago) Jun 17

to Dominic Farolino, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Dominic Farolino

Fr added 6 comments

File third_party/blink/renderer/platform/runtime_enabled_features.json5

Line 6866, Patchset 5 (Parent): name: "WebMCPTesting",

Fr . resolved

If we're removing this feature, it means we'll have to update [Puppeteer](https://pptr.dev/guides/webmcp), [Chrome DevTools for Agent](https://github.com/ChromeDevTools/chrome-devtools-mcp), and maybe more documentation. Is there a way somehow to keep this functional?

Dominic Farolino

I can hold off on removing it for now, but we should remove it soon. Can you kick off the process of changing whatever you'd want to change for us to be able to remove this? I'll try and remember to remove it in the next milestone.

Fr

I'll start this process. Thanks!

File third_party/blink/web_tests/external/wpt/webmcp/declarative/custom-element-fill.https.html

Line 2, Patchset 8 (Latest):<title>WebMCP: Filling custom element</title>

Fr . unresolved

I don't think this should be moved to external since `toolFillCallback` is not spec'ed and still highly experimental

Line 47, Patchset 8 (Latest): await waitForTool('fill_text');

Fr . unresolved

How about we update `waitForTool` to return the Tool from getTools so that we don't have to run `document.modelContext.getTools` again?

File third_party/blink/web_tests/external/wpt/webmcp/imperative/executeTool-invalid-dictionary.https.html

Line 40, Patchset 8 (Latest): document.modelContext.registerTool({

Fr . unresolved

Shall we add `await` now before `registerTool()`?

File third_party/blink/web_tests/inspector-protocol/webmcp/log-tool-calls.js

Line 117, Patchset 8 (Latest): const tool = await getTool("failing_js_tool");

Fr . unresolved

This is not valid.

File third_party/blink/web_tests/virtual/webmcp/inspector-protocol/webmcp/log-tool-calls-expected.txt

Line 143, Patchset 8 (Latest): output : {

Fr . unresolved

Is this change expected?

Open in Gerrit

Related details

Attention is currently required from:

Dominic Farolino

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review

No-Unresolved-Comments

Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Dominic Farolino (Gerrit)

unread,

Jun 17, 2026, 11:05:17 AM (11 days ago) Jun 17

to kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, Fr, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Fr

Dominic Farolino voted and added 5 comments

Votes added by Dominic Farolino

Commit-Queue

+1

5 comments

File third_party/blink/web_tests/external/wpt/webmcp/declarative/custom-element-fill.https.html

Line 2, Patchset 8:<title>WebMCP: Filling custom element</title>

Fr . resolved

I don't think this should be moved to external since `toolFillCallback` is not spec'ed and still highly experimental

Dominic Farolino

Done

Line 47, Patchset 8: await waitForTool('fill_text');

Fr . resolved

How about we update `waitForTool` to return the Tool from getTools so that we don't have to run `document.modelContext.getTools` again?

Dominic Farolino

Nice. Done.

File third_party/blink/web_tests/external/wpt/webmcp/imperative/executeTool-invalid-dictionary.https.html

Line 40, Patchset 8: document.modelContext.registerTool({

Fr . resolved

Shall we add `await` now before `registerTool()`?

Dominic Farolino

Not before https://crrev.com/c/7916507 has landed, right?

File third_party/blink/web_tests/inspector-protocol/webmcp/log-tool-calls.js

Line 117, Patchset 8: const tool = await getTool("failing_js_tool");

Fr . unresolved

This is not valid.

Dominic Farolino

Can you clarify what isn't valid? Is this helper incorrect, or something else?

File third_party/blink/web_tests/virtual/webmcp/inspector-protocol/webmcp/log-tool-calls-expected.txt

Line 143, Patchset 8: output : {

Fr . unresolved

Is this change expected?

Dominic Farolino

Unfortunately yes. ModelContextTesting supports tool cancellation via AbortSignal [here](https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/script_tools/model_context.cc;l=629-631;drc=13c5969778bff936fb3991389511f695fd2e0319), which previously let us surface `status: Canceled` here. But the ordinary web platform API doesn't yet support cancellation through signal (see http://b/481899636), which means `ScriptToolErrorCode::kToolCancelled` is never communicated, which means the inspector never [hits this path](https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/inspector/inspector_web_mcp_agent.cc;l=230-232;drc=6ce6f0f91493b5c3a3ce713a2ef454db5deb45a3).

So this expectation reflects the migration to the normal web platform API which doesn't really support tool execution cancellation, and this expectation will be reverted once we fix the above bug.

Does this make sense?

Open in Gerrit

Related details

Attention is currently required from:

Fr

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
No-Unresolved-Comments
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Fr (Gerrit)

unread,

Jun 17, 2026, 11:22:05 AM (11 days ago) Jun 17

to Dominic Farolino, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Dominic Farolino

Fr voted and added 6 comments

Votes added by Fr

Code-Review

+1

6 comments

Patchset-level comments

File-level comment, Patchset 9 (Latest):

Fr . resolved

LGTM with nits

File third_party/blink/web_tests/external/wpt/webmcp/imperative/executeTool-invalid-dictionary.https.html

Line 40, Patchset 8: document.modelContext.registerTool({

Fr . resolved

Shall we add `await` now before `registerTool()`?

Dominic Farolino

Not before https://crrev.com/c/7916507 has landed, right?

Fr

I thought we could add `await` since it's spec compliant AND it works now, even without the implementation. If not, I'll have to update those tests when adding promise support to registerTool

File third_party/blink/web_tests/inspector-protocol/webmcp/log-tool-calls.js

Line 117, Patchset 8: const tool = await getTool("failing_js_tool");

Fr . resolved

This is not valid.

Dominic Farolino

Can you clarify what isn't valid? Is this helper incorrect, or something else?

Fr

I got confused by the tool name. Sorry for the noise.

File third_party/blink/web_tests/virtual/webmcp/inspector-protocol/webmcp/log-tool-calls-expected.txt

Line 114, Patchset 9 (Parent):

        description : Error: JS Error     at failing_js (<anonymous>:33:48)     at window.executeFailingJS (<anonymous>:79:49)     at <anonymous>:1:8

Fr . unresolved

Do you know what caused this line change?

Line 143, Patchset 8: output : {

Fr . resolved

Is this change expected?

Dominic Farolino

Unfortunately yes. ModelContextTesting supports tool cancellation via AbortSignal [here](https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/script_tools/model_context.cc;l=629-631;drc=13c5969778bff936fb3991389511f695fd2e0319), which previously let us surface `status: Canceled` here. But the ordinary web platform API doesn't yet support cancellation through signal (see http://b/481899636), which means `ScriptToolErrorCode::kToolCancelled` is never communicated, which means the inspector never [hits this path](https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/core/inspector/inspector_web_mcp_agent.cc;l=230-232;drc=6ce6f0f91493b5c3a3ce713a2ef454db5deb45a3).
So this expectation reflects the migration to the normal web platform API which doesn't really support tool execution cancellation, and this expectation will be reverted once we fix the above bug.
Does this make sense?

Fr

I didn't realize `modelContext.executeTool(myTool, { signal })` was not supported yet. Thanks for the details!

File third_party/blink/web_tests/wpt_internal/webmcp/custom-element-fill.https.html

Line 48, Patchset 9 (Latest): assert_true(!!tool, 'Tool should be registered');

Fr . unresolved

Why `assert_true`? `waitForTool` won't resolve if there's no tool.

Open in Gerrit

Related details

Attention is currently required from:

Dominic Farolino

Submit Requirements:

Code-Coverage
Code-Owners

Code-Review

No-Unresolved-Comments

Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Dominic Farolino (Gerrit)

unread,

Jun 17, 2026, 1:42:50 PM (11 days ago) Jun 17

to Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Dominic Farolino voted and added 2 comments

Votes added by Dominic Farolino

Commit-Queue

+2

2 comments

File third_party/blink/web_tests/virtual/webmcp/inspector-protocol/webmcp/log-tool-calls-expected.txt

Line 114, Patchset 9 (Parent):

        description : Error: JS Error     at failing_js (<anonymous>:33:48)     at window.executeFailingJS (<anonymous>:79:49)     at <anonymous>:1:8

Fr . resolved

Do you know what caused this line change?

Dominic Farolino

Yeah, with `ModelContextTesting`, `executeTool()` invoked the local tool synchronously inside `window.executeFailingJS()`. So the call stack included at window.executeFailingJS. With document.modelContext, execution happens asynchronously over Mojo, so `failing_js()` executes outside of the synchronous `executeFailingJS()` calling function. So `window.executeFailingJS` is naturally absent from the stack trace. Does that make sense?

I'll resolve this for now since I think this answers it, and we can land this. But if you want more clarity lmk!

File third_party/blink/web_tests/wpt_internal/webmcp/custom-element-fill.https.html

Line 48, Patchset 9: assert_true(!!tool, 'Tool should be registered');

Fr . resolved

Why `assert_true`? `waitForTool` won't resolve if there's no tool.

Dominic Farolino

True, removed!

Open in Gerrit

Related details

Attention set is empty

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review

Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Dominic Farolino (Gerrit)

unread,

Jun 17, 2026, 1:44:03 PM (11 days ago) Jun 17

to Alex Moshchuk, Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Alex Moshchuk

Dominic Farolino voted and added 1 comment

Votes added by Dominic Farolino

Auto-Submit	+1
Commit-Queue	+1

1 comment

Patchset-level comments

File-level comment, Patchset 10 (Latest):

Dominic Farolino . resolved

Alex, could you look at `model_context_user_data.cc`? The change there is pretty small.

Open in Gerrit

Related details

Attention is currently required from:

Alex Moshchuk

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

unsatisfied_requirement

open

diffy

Alex Moshchuk (Gerrit)

unread,

Jun 17, 2026, 2:14:33 PM (11 days ago) Jun 17

to Dominic Farolino, Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Attention needed from Dominic Farolino

Alex Moshchuk voted and added 1 comment

Votes added by Alex Moshchuk

Code-Review

+1

1 comment

Patchset-level comments

File-level comment, Patchset 10 (Latest):

Dominic Farolino . resolved

Alex, could you look at `model_context_user_data.cc`? The change there is pretty small.

Alex Moshchuk

That file LGTM

Open in Gerrit

Related details

Attention is currently required from:

Dominic Farolino

Submit Requirements:

Code-Coverage
Code-Owners

Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Dominic Farolino (Gerrit)

unread,

Jun 17, 2026, 2:17:03 PM (11 days ago) Jun 17

to Alex Moshchuk, Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Dominic Farolino voted Commit-Queue+2

Commit-Queue

+2

Open in Gerrit

Related details

Attention set is empty

satisfied_requirement

open

diffy

Blink W3C Test Autoroller (Gerrit)

unread,

Jun 17, 2026, 2:27:09 PM (11 days ago) Jun 17

to Dominic Farolino, Alex Moshchuk, Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, Chromium LUCI CQ, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Message from Blink W3C Test Autoroller

Exportable changes to web-platform-tests were detected in this CL and a pull request in the upstream repo has been made: https://github.com/web-platform-tests/wpt/pull/60698.

When this CL lands, the bot will automatically merge the PR on GitHub if the required GitHub checks pass; otherwise, ecosystem-infra@ team will triage the failures and may contact you.

WPT Export docs:
https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md#Automatic-export-process

Open in Gerrit

Related details

Attention set is empty

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Chromium LUCI CQ (Gerrit)

unread,

Jun 17, 2026, 3:18:20 PM (11 days ago) Jun 17

to Dominic Farolino, Blink W3C Test Autoroller, Alex Moshchuk, Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Chromium LUCI CQ submitted the change

Change information

Commit message:

WebMCP: Remove the ModelContextTesting API

R=beaufort...@gmail.com

Bug: 515272366

Change-Id: If259e8c5194b83504401a937ccc366c16ccd5ef3

Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7895790

Reviewed-by: Fr <beaufort...@gmail.com>

Auto-Submit: Dominic Farolino <d...@chromium.org>

Reviewed-by: Alex Moshchuk <ale...@chromium.org>

Commit-Queue: Dominic Farolino <d...@chromium.org>

Cr-Commit-Position: refs/heads/main@{#1648512}

Files:

M android_webview/test/data/web_tests/webexposed/global-interface-listing-expected.txt
M chrome/browser/about_flags.cc
M chrome/browser/flag_descriptions.h
M content/browser/renderer_host/model_context_user_data.cc
M third_party/blink/renderer/bindings/generated_in_core.gni
M third_party/blink/renderer/bindings/idl_in_core.gni
M third_party/blink/renderer/core/script_tools/build.gni
M third_party/blink/renderer/core/script_tools/model_context.cc
M third_party/blink/renderer/core/script_tools/model_context.h
M third_party/blink/renderer/core/script_tools/model_context.idl
M third_party/blink/renderer/core/script_tools/model_context_supplement.cc
M third_party/blink/renderer/core/script_tools/model_context_supplement.h
M third_party/blink/renderer/core/script_tools/model_context_supplement.idl
M third_party/blink/renderer/core/script_tools/model_context_test.cc
D third_party/blink/renderer/core/script_tools/model_context_testing.cc
D third_party/blink/renderer/core/script_tools/model_context_testing.h
D third_party/blink/renderer/core/script_tools/model_context_testing.idl
R third_party/blink/web_tests/external/wpt/webmcp/declarative/execute_tool_change_event.https.html
R third_party/blink/web_tests/external/wpt/webmcp/declarative/execute_tool_submit_from_js.https.html
R third_party/blink/web_tests/external/wpt/webmcp/declarative/form_removal_submit_crash.https.html
R third_party/blink/web_tests/external/wpt/webmcp/declarative/select-multiple-events.https.html
M third_party/blink/web_tests/external/wpt/webmcp/declarative/toolchange-on-attribute-mutation.https.html
M third_party/blink/web_tests/external/wpt/webmcp/imperative/executeTool-invalid-dictionary.https.html
R third_party/blink/web_tests/external/wpt/webmcp/imperative/initial-about-blank-shared-tool.https.html
M third_party/blink/web_tests/external/wpt/webmcp/resources/helpers.js
M third_party/blink/web_tests/fast/dom/Window/property-access-on-cached-properties-after-frame-navigated-expected.txt
M third_party/blink/web_tests/fast/dom/Window/property-access-on-cached-properties-after-frame-removed-and-gced-expected.txt
M third_party/blink/web_tests/fast/dom/Window/property-access-on-cached-properties-after-frame-removed-expected.txt
M third_party/blink/web_tests/fast/dom/Window/property-access-on-cached-window-after-frame-navigated-expected.txt
D third_party/blink/web_tests/fast/webmcp/execute_tool-expected.txt
D third_party/blink/web_tests/fast/webmcp/execute_tool.html
D third_party/blink/web_tests/fast/webmcp/get_cross_document_script_tool_result.https-expected.txt
D third_party/blink/web_tests/fast/webmcp/get_cross_document_script_tool_result.https.html
D third_party/blink/web_tests/fast/webmcp/list_tools-expected.txt
D third_party/blink/web_tests/fast/webmcp/list_tools.html
D third_party/blink/web_tests/fast/webmcp/resources/get_cross_document_script_tool_result_target.html
D third_party/blink/web_tests/fast/webmcp/toolchange.html
M third_party/blink/web_tests/fast/webmcp/usecounter.https.html
M third_party/blink/web_tests/inspector-protocol/model-context/missing-parameter-title-description-issue.js
M third_party/blink/web_tests/inspector-protocol/webmcp/execute-declarative-tool-crash.js
M third_party/blink/web_tests/inspector-protocol/webmcp/log-tool-calls.js
M third_party/blink/web_tests/virtual/webmcp/inspector-protocol/webmcp/log-tool-calls-expected.txt
M third_party/blink/web_tests/webexposed/global-interface-listing-expected.txt
M third_party/blink/web_tests/wpt_internal/webmcp/custom-element-fill.https.html
M third_party/blink/web_tests/wpt_internal/webmcp/custom-element-invalid-schema.https.html
M tools/metrics/histograms/enums.xml

Change size: L

Delta: 46 files changed, 161 insertions(+), 809 deletions(-)

Branch: refs/heads/main

Submit Requirements:

Code-Review: +1 by Fr, +1 by Alex Moshchuk

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

open

diffy

satisfied_requirement

Blink W3C Test Autoroller (Gerrit)

unread,

Jun 17, 2026, 4:41:53 PM (11 days ago) Jun 17

to Dominic Farolino, Chromium LUCI CQ, Alex Moshchuk, Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Message from Blink W3C Test Autoroller

The WPT PR for this CL has been merged upstream! https://github.com/web-platform-tests/wpt/pull/60698

Open in Gerrit

Related details

Attention set is empty

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Reilly Grant (Gerrit)

unread,

Jun 17, 2026, 5:18:10 PM (11 days ago) Jun 17

to Dominic Farolino, Chromium LUCI CQ, Blink W3C Test Autoroller, Alex Moshchuk, Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org, Reilly Grant

Reilly Grant has created a revert of this change

Open original change

Related details

Attention set is empty

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Magdalena Mielnicka (Gerrit)

unread,

Jun 22, 2026, 12:13:42 AM (7 days ago) Jun 22

to Dominic Farolino, Chromium LUCI CQ, Blink W3C Test Autoroller, Alex Moshchuk, Fr, kokoro, Chromium Metrics Reviews, Menard, Alexis, android-bu...@system.gserviceaccount.com, chromium...@chromium.org, Kentaro Hara, Khushal Sagar, Raphael Kubo da Costa, asvitkine...@chromium.org, navigation...@chromium.org, apavlo...@chromium.org, alexmo...@chromium.org, creis...@chromium.org, blink-re...@chromium.org, asvitki...@chromium.org, kinuko...@chromium.org, android-web...@chromium.org, ashleynewson+watch-...@chromium.org, blink-revie...@chromium.org, blink-revie...@chromium.org, blink-...@chromium.org, devtools-re...@chromium.org, jmedle...@chromium.org, mfoltz+wa...@chromium.org

Magdalena Mielnicka added 1 comment

Patchset-level comments

File-level comment, Patchset 11 (Latest):

Magdalena Mielnicka . resolved

magdalena.m...@gmail.com

Open in Gerrit

Related details

Attention set is empty

Submit Requirements:

Code-Coverage
Code-Owners
Code-Review
Review-Enforcement

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

satisfied_requirement

open

diffy

Reply all

Reply to author

Forward

WebMCP: Remove the ModelContextTesting API [chromium/src : main]

Dominic Farolino (Gerrit)

Dominic Farolino voted and added 5 comments

Votes added by Dominic Farolino

5 comments

Related details

Dominic Farolino (Gerrit)

Dominic Farolino voted Commit-Queue+1

Related details

Fr (Gerrit)

Fr added 6 comments

Related details

Dominic Farolino (Gerrit)

Dominic Farolino voted and added 5 comments

Votes added by Dominic Farolino

5 comments

Related details

Fr (Gerrit)

Fr voted and added 6 comments

Votes added by Fr

6 comments

Related details

Dominic Farolino (Gerrit)

Dominic Farolino voted and added 2 comments

Votes added by Dominic Farolino

2 comments

Related details

Dominic Farolino (Gerrit)

Dominic Farolino voted and added 1 comment

Votes added by Dominic Farolino

1 comment

Related details

Alex Moshchuk (Gerrit)

Alex Moshchuk voted and added 1 comment

Votes added by Alex Moshchuk

1 comment

Related details

Dominic Farolino (Gerrit)

Dominic Farolino voted Commit-Queue+2

Related details

Blink W3C Test Autoroller (Gerrit)

Message from Blink W3C Test Autoroller

Related details

Chromium LUCI CQ (Gerrit)

Chromium LUCI CQ submitted the change

Change information

Blink W3C Test Autoroller (Gerrit)

Message from Blink W3C Test Autoroller

Related details

Reilly Grant (Gerrit)

Reilly Grant has created a revert of this change

Related details

Magdalena Mielnicka (Gerrit)

Magdalena Mielnicka added 1 comment

Related details