[WebMCP] Add ModelContext features to UKM [chromium/src : main]

0 views
Skip to first unread message

Fr (Gerrit)

unread,
1:58 AM (2 hours ago) 1:58 AM
to Ben Greenstein, Ryan Sturm, Dominic Farolino, Robert Kaplow, android-bu...@system.gserviceaccount.com, chromium...@chromium.org, bmcquad...@chromium.org, csharris...@chromium.org, loading-rev...@chromium.org, speed-metrics...@chromium.org, speed-metr...@chromium.org
Attention needed from Ben Greenstein, Dominic Farolino, Robert Kaplow and Ryan Sturm

Fr added 1 comment

File components/page_load_metrics/browser/observers/use_counter/ukm_features.cc
Line 484, Patchset 1 (Latest): WebFeature::kModelContextRegisterTool,
Fr . unresolved

Why not `kModelContextGetTools` as well?

Open in Gerrit

Related details

Attention is currently required from:
  • Ben Greenstein
  • Dominic Farolino
  • Robert Kaplow
  • Ryan Sturm
Submit Requirements:
  • requirement satisfiedCode-Coverage
  • requirement is not satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedNo-Unresolved-Comments
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: chromium/src
Gerrit-Branch: main
Gerrit-Change-Id: I31b418f8a4f88346bfee8ad36d4fc16534665b06
Gerrit-Change-Number: 8145499
Gerrit-PatchSet: 1
Gerrit-Owner: Ben Greenstein <be...@chromium.org>
Gerrit-Reviewer: Dominic Farolino <d...@chromium.org>
Gerrit-Reviewer: Robert Kaplow <rka...@chromium.org>
Gerrit-Reviewer: Ryan Sturm <ryan...@chromium.org>
Gerrit-CC: Fr <beaufort...@gmail.com>
Gerrit-Attention: Ben Greenstein <be...@chromium.org>
Gerrit-Attention: Dominic Farolino <d...@chromium.org>
Gerrit-Attention: Robert Kaplow <rka...@chromium.org>
Gerrit-Attention: Ryan Sturm <ryan...@chromium.org>
Gerrit-Comment-Date: Sat, 25 Jul 2026 05:57:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
satisfied_requirement
unsatisfied_requirement
open
diffy
Reply all
Reply to author
Forward
0 new messages